Skip to content
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

WingetUI installs user setup instead of upgrading system installation on upgrade #142

Closed
sitiom opened this issue Sep 17, 2022 · 12 comments
Closed
Labels
bug Something isn't working needs-author-answer

Comments

@sitiom
Copy link
Contributor

sitiom commented Sep 17, 2022

This bug is quite annoying.

Steps to Reproduce

  1. Install an old version of VSCode in the machine scope: winget install Microsoft.VisualStudioCode --scope machine -v <old version>
  2. Upgrade VSCode via WingetUI
  3. WingetUI installs the user setup of VSCode, leaving with two installations of VSCode (user and machine wide)

Alternatively, upgrading via the commandline (winget upgrade Microsoft.VisualStudioCode) properly upgrades VSCode.

@marticliment
Copy link
Owner

Mhmm... Which version of winget are you running on your local machine?

@marticliment marticliment added bug Something isn't working needs-author-answer labels Sep 17, 2022
@sitiom
Copy link
Contributor Author

sitiom commented Sep 17, 2022

v1.3.2091

@Satanarious
Copy link
Contributor

Same for me

image

@ppvnf
Copy link
Contributor

ppvnf commented Jan 3, 2023

I believe this was fixed by #575

@marticliment
Copy link
Owner

It should. I'll test it further but it should

@sitiom
Copy link
Contributor Author

sitiom commented Jan 3, 2023

Tested the latest commit, same thing. Here is from the live output:
image

User setup is installed.

Expected output:
image

@ppvnf
Copy link
Contributor

ppvnf commented Jan 3, 2023

@sitiom do you still get two separate vscode installs after using WingetUI to update?

@sitiom
Copy link
Contributor Author

sitiom commented Jan 3, 2023

@sitiom do you still get two separate vscode installs after using WingetUI to update?

Yes

@marticliment
Copy link
Owner

@sitiom, which version of winget are you running on your system?

@sitiom
Copy link
Contributor Author

sitiom commented Jan 3, 2023

v1.3.2691

@marticliment
Copy link
Owner

Could you please test it now, with the latest commit?

@sitiom
Copy link
Contributor Author

sitiom commented Jan 3, 2023

Now it works, thanks.

@sitiom sitiom closed this as completed Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-author-answer
Projects
None yet
Development

No branches or pull requests

4 participants