Sitelet https://web.archive.org/web/20210120041556/https://github.com/gerardog/gsudo/issues/17
Skip to content

/ gsudo

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

Jump to bottom

`gsudo chocolatey upgrade gsudo` ends up breaking gsudo command #17

Closed
lostmsu opened this issue Feb 1, 2020 · 1 comment
Closed

`gsudo chocolatey upgrade gsudo` ends up breaking gsudo command #17

lostmsu opened this issue Feb 1, 2020 · 1 comment

Comments

@lostmsu
Copy link

@lostmsu lostmsu commented Feb 1, 2020 •
edited by gerardog

Looks like the install script kills gsudo, that runs it. 😂
Workaround is to remove C:\ProgramData\chocolatey\bin\gsudo.exe (which is a link) and to reinstall gsudo from a regular elevated console.
@gerardog
Copy link
Owner

@gerardog gerardog commented Feb 4, 2020 •
edited

Using gsudo to upgrade gsudo is not supported right now. Please open and use a CMD as administrator. ¯_(ツ)_/¯

Under some circumstances, surprisingly, it works. 😁

But in your case, you went into what's described in #18, not strictly caused by using gsudo in the command, but a bug because I didnt test the choco upgrade scenario correctly. Still, you used the expected workaround! 🎉

Nonetheless, I will leave this issue open until, hopefully the scripts properly detects and stops the upgrade if gsudo is open, which I am not completely sure if it's possible (chocolatey/choco#1731).
Or, at least, it shows a big warning message.
@gerardog gerardog mentioned this issue Apr 2, 2020
Merged
@gerardog gerardog closed this in f0ae278 Apr 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@lostmsu @gerardog
You can’t perform that action at this time.