-
Notifications
You must be signed in to change notification settings - Fork 410
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
BUG: pipx upgrade doesn't understand extra dependencies #925
Comments
You can simply run |
Does |
Not really, unless |
Even if we change the strategy, dependencies in the extra would still not be upgraded (because from pip’s perspective those packages are unrelated unless you pass in the extra). So for now |
PR welcome. |
This might've been fixed since? It looks like if I |
What should pipx do if the main app's version didn't change? Should it still upgrade dependencies? (IMO it should. This will let users benefit from bugfixes and security updates to the app's dependency tree. It will also help with the more exotic case of |
Describe the bug
This works correctly
but this doesn't
and it fails with
How to reproduce
For reproduction, I suggest
Expected behavior
I'm not 100% sure what should happen actually, but the current error message seems broken (of course a directory with
[
and]
in its name wasn't found).I think the upgrade should go through wether or not the extra targets match the ones I used (or didn't use) the first time I installed the CLI (here
coverage
).I'd be happy to contribute a patch, but I'd appreciate some guidance regarding what the expected behaviour should be.
The text was updated successfully, but these errors were encountered: