-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmd: ipfs handle GUI environment on Windows #6646
Conversation
I'm thinking this should also pass Opinions? |
Nice! I'm slightly concerned that we'll log something on init and break this but I can't think of a good way to fix that.
👍. This makes sense. We've been discussing making this the default anyways. |
cee93bd
to
ff488b2
Compare
Force pushed over this. |
We should probably explicitly block on #1897 I'll have to see if there's a way to get notified of these events from the OS. Otherwise we won't gracefully shutdown (unless the user sends ctrl-c/interrupt). |
This is no longer the case in Go 1.14. |
ff488b2
to
3c9039a
Compare
❤️ |
Resolves: #6633
Hacky but maybe acceptable.
If we're not launched from a terminal, we launch the daemon by default.
If we're inside of a terminal, we print help.
https://youtu.be/diam0dQaBdw
(for some reason my cursor didn't get recorded)