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

Implement Uninstall [URL] #4958

Open
jidanni opened this issue Jan 6, 2018 · 4 comments
Open

Implement Uninstall [URL] #4958

jidanni opened this issue Jan 6, 2018 · 4 comments
Labels
state: awaiting PR Feature discussed, PR is needed type: enhancement Improvements to functionality

Comments

@jidanni
Copy link

jidanni commented Jan 6, 2018

Please make this a clean roundabout trip, leaving the system the way it was

# pip   install https://electrum-ltc.org/download/Electrum-LTC-2.9.3.1.tar.gz
# pip uninstall https://electrum-ltc.org/download/Electrum-LTC-2.9.3.1.tar.gz

or at least have it raise an error.

@pradyunsg
Copy link
Member

While I don't think this should start working or raising an error, I think having pip uninstall print a warning when given something that doesn't look like a package name (like a URL or directory path) is a reasonable thing to do.

@pradyunsg pradyunsg added type: enhancement Improvements to functionality state: awaiting PR Feature discussed, PR is needed labels Jan 26, 2018
@pradyunsg
Copy link
Member

I've labelled this issue as an "awaiting PR".

This label is essentially for indicating that further discussion related to this issue should be deferred until someone comes around to make a PR. This does not mean that the said PR would be accepted - it has not been determined whether this is a useful change to pip and that decision has been deferred until the PR is made.

@chrahunt
Copy link
Member

chrahunt commented Dec 9, 2019

Related to PEP 610 which, if it gets accepted, would make this easier since we could do an installed package metadata query as opposed to downloading the archive, generating its metadata, then querying for an existing package matching it.

@GuyTuval
Copy link
Contributor

GuyTuval commented Oct 28, 2020

Hi @jidanni , @pradyunsg !
I have created a pull request addressing this issue.
Hope it'll help progress the discussion and if any change needs to be made. you are more than welcome to let me know 😁
PR:
Handle invalid parameters to uninstall

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
state: awaiting PR Feature discussed, PR is needed type: enhancement Improvements to functionality
Projects
None yet
Development

No branches or pull requests

4 participants