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

Cannot Uninstall MiKTeX #1711

Closed
keenbowl3009 opened this issue Nov 15, 2021 · 14 comments
Closed

Cannot Uninstall MiKTeX #1711

keenbowl3009 opened this issue Nov 15, 2021 · 14 comments
Labels
Issue-Bug It either shouldn't be doing this or needs an investigation.
Milestone

Comments

@keenbowl3009
Copy link

keenbowl3009 commented Nov 15, 2021

Brief description of your issue

Cannot uninstall the package using the package Id.

Steps to reproduce

winget install ChristianSchenk.MiKTeX
winget uninstall ChristianSchenk.MiKTeX

Expected behavior

The system should return exactly to the state it was if miktex was not installed.

Actual behavior

When trying to uninstall, I get:
No installed package found matching input criteria.

Environment

Windows: Windows.Desktop v10.0.22000.318
Package: Microsoft.DesktopAppInstaller v1.16.12663.0

Edited: Off topic comments removed.

@ghost ghost added the Needs-Triage Issue need to be triaged label Nov 15, 2021
@jthoward64
Copy link

jthoward64 commented Nov 15, 2021

What is the output of winget list
I'm not encouraging this guy

@keenbowl3009
Copy link
Author

What is the output of winget list

the output of winget list includes MiKTeX.

@keenbowl3009
Copy link
Author

The output of winget uninstall ChristianSchenk.MiKTeX is:

No installed package found matching input criteria.

@keenbowl3009

This comment has been minimized.

@keenbowl3009

This comment has been minimized.

@JanPokorny
Copy link

JanPokorny commented Nov 15, 2021

I was able to reproduce. MiKTeX has wrong id after installation:

image

image

It needs to be uninstalled with winget uninstall MiKTeX:
image

@Masamune3210
Copy link

Manifest probably has incorrect productid or they changed it recently as I think that is what winget uses currently to detect known installed applications

@jedieaston
Copy link
Contributor

jedieaston commented Nov 15, 2021

This will most likely be resolved by microsoft/winget-pkgs#34983. Once it merges you should be good to go.

@martinmine
Copy link

winget is literally the worst possible package manager that the community could adopt.

Your attempt at shitposting in order to let out your frustration will not be any helpful for improving this software. Your issues might even get deleted as they clearly violate the code of conduct. If winget upsets you that much you create a github account specifically to troll on this repo, I seriously recommend you that you just use Scoop as that is a much better usage of your time.

@JanPokorny
Copy link

@martinmine The issue is actually real. The autor probably isn't, tho.

@martinmine
Copy link

@JanPokorny Sure, but the tone and content isn't constructive and is only helpful for OP to let out their frustration. This is counter-productive and if people actually have this issue, now there is the risk it might even get deleted.

@denelon denelon added Issue-Bug It either shouldn't be doing this or needs an investigation. and removed Needs-Triage Issue need to be triaged labels Nov 15, 2021
@denelon
Copy link
Contributor

denelon commented Nov 15, 2021

PR merged. Once it's been published, this should be resolved.

@denelon denelon closed this as completed Nov 16, 2021
@JanPokorny
Copy link

I feel sorry for Microsoft open source maintainers that have to deal with this sort of crazy...

@denelon
Copy link
Contributor

denelon commented Nov 17, 2021

@JanPokorny it comes with the territory. People have a long memory. It's going to take even longer to demonstrate how much we care about open source. I understand the frustration here. As long as it's not directed towards an individual, it's better to let people vent in most cases. There are also several different perspectives on the "right" approach when there are many right approaches.

Most of the community has been very supportive which makes it easier to want to build a better product and be honest about failures and gaps. There is a massive body of work in the world of Linux package management where everything is a file. Things are far more complex when dealing with third party installers and decades of inertia. It's an exciting time to be at Microsoft and have the opportunity to be open about what we're building, and get the customer feedback early and often.

Thank you for your support 😊

@denelon denelon added this to the v1.3-Client milestone Jun 21, 2022
Trenly added a commit to Trenly/winget-cli that referenced this issue Mar 22, 2024
411a10915 mint 2.10.19
096a9a1bb Merge pull request microsoft#1782 from microsoft/fix_uri_parsing
006271f6a make Uri.is_host_loopback() only return true for localhost and 127.0.0.1 exactly
9c654889e Remove email list from the readme
31e7feacc Merge pull request microsoft#1718 from dashanji/Fix-typo
e1b6a8e61 Merge pull request microsoft#1711 from Fighter19/pr-fix-safeint3
c5dcbb5bb Merge pull request microsoft#1717 from microsoft/users/GitHubPolicyService/f2ee14d6-8d6b-4313-b754-3880e015f7c3
98ee36dac Fix typo
a57f45918 Microsoft mandatory file
3308d9728 Fix likely typo in SafeInt3.hpp, that results in error with clang 15
07cf58910 Merge pull request microsoft#1429 from NN---/fix/value_int_ctor
3eac925ad Update Release/include/cpprest/json.h
06363bc78 Update Release/include/cpprest/json.h
bfe348779 Merge pull request #1577 from JvdGlind/hidden_visibility_support_macos
0ddc61829 Clarify cpprestsdk level of support
804448058 export http_exception for non Windows builds
d9d7f5ed4 Merge pull request microsoft#1496 from icherniukh/oauth2_client_credentials
8ae5da616 Update oauth2.h
708a5df2b Add support for oauth2 using only client credentials
5408f1dc9 Add constructor from all integer types.

git-subtree-dir: src/cpprestsdk/cpprestsdk
git-subtree-split: 411a109150b270f23c8c97fa4ec9a0a4a98cdecf
Trenly added a commit to Trenly/winget-cli that referenced this issue Mar 22, 2024
411a10915 mint 2.10.19
096a9a1bb Merge pull request microsoft#1782 from microsoft/fix_uri_parsing
006271f6a make Uri.is_host_loopback() only return true for localhost and 127.0.0.1 exactly
9c654889e Remove email list from the readme
31e7feacc Merge pull request microsoft#1718 from dashanji/Fix-typo
e1b6a8e61 Merge pull request microsoft#1711 from Fighter19/pr-fix-safeint3
c5dcbb5bb Merge pull request microsoft#1717 from microsoft/users/GitHubPolicyService/f2ee14d6-8d6b-4313-b754-3880e015f7c3
98ee36dac Fix typo
a57f45918 Microsoft mandatory file
3308d9728 Fix likely typo in SafeInt3.hpp, that results in error with clang 15
07cf58910 Merge pull request microsoft#1429 from NN---/fix/value_int_ctor
3eac925ad Update Release/include/cpprest/json.h
06363bc78 Update Release/include/cpprest/json.h
bfe348779 Merge pull request #1577 from JvdGlind/hidden_visibility_support_macos
0ddc61829 Clarify cpprestsdk level of support
804448058 export http_exception for non Windows builds
d9d7f5ed4 Merge pull request microsoft#1496 from icherniukh/oauth2_client_credentials
8ae5da616 Update oauth2.h
708a5df2b Add support for oauth2 using only client credentials
5408f1dc9 Add constructor from all integer types.

git-subtree-dir: src/cpprestsdk/cpprestsdk
git-subtree-split: 411a109150b270f23c8c97fa4ec9a0a4a98cdecf
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Bug It either shouldn't be doing this or needs an investigation.
Projects
None yet
Development

No branches or pull requests

8 participants
@Masamune3210 @martinmine @JanPokorny @jedieaston @jthoward64 @denelon @keenbowl3009 and others