-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Pip isn't accounting for Setuptool's requirement extras #7
Comments
Hmm, no comment? Is supporting this planned? Is its missing considered a bug? Is there some
|
This is also an issue for me. I started digging into the pip code a little
|
maybe this will do it:
|
Made some adjustments to my proposed patch, see see https://bitbucket.org/MattMaker/pip/changeset/4a967b7de7ad and https://bitbucket.org/MattMaker/pip/changeset/9c681d36b768 . Now it works with uninstalling too! (oops :) ) |
Matthew - can you fork the new pip repo on GitHub and send a pull request for your changes? Thanks |
forked, retested, and pull requested! |
Fixed. Thanks @MatthewMaker for the patch, and @ssaboum for adding tests. |
Just a quick question about this... I don't think it is documented at all and I think it should be, probably in the requirements.txt section. My question is.. does this work in all places where you could use install before? I usually tested things out locally by running I added this to my setup.py:
... but now I don't see how I could install it. I tried |
@poswald Thanks for noting the lack of documentation - if you'd be willing to open an issue for that (or better, a pull request!) that'd be great. As for the format, using |
…additional dependencies. See: Issue pypa#7
There's the documentation for it in the requirements.txt file. I only use the I'll play around and open any issues if I see it isn't working as expected. Thanks! |
Oh - I was under the impression from your previous comment that |
I just didn't know how to properly specify it for a local install to test it because I didn't know how to do |
Sorry guys, I may have spoken too soon on this. I thought it was working before but I must have done something wrong as doesn't seem to be. Here I have a project with an
Assuming I did that correctly, it seems that the issue is that the line to unpack the Is there an equivalent to Should this be a separate issue or perhaps should this one be reopened? |
Whatever needs to be done here, if anything, should be a separate issue. Extras work just fine for the common case (installing "Foo[Bar]" off PyPI) which solves this ticket; I didn't really ever expect it to work for other cases. If it's possible to make it work, that's great, but it's a separate issue. Based on reading the code, I didn't expect it to work using the syntax you tried, I expect that it should work if you |
Found that you can use extra req when installing from URL. pypa/pip#7
…toto.always-verify-targets if switched on, always verify in-toto metadata, regardless of TUF update
Pip's requirements parsing doesn't appear to recognize declared extras.[0] These are supposed to be used in Setuptools to provide a way to list dependencies for a particular feature.
Repeatable through the following:
In the above 'zope.app.testing' should have been pulled in as a dependency, because of the extra declaration in the requirements line.
Another example would be "zope.i18n [zcml]".
[0] http://peak.telecommunity.com/DevCenter/setuptools#declaring-extras-optional-features-with-their-own-dependencies
The text was updated successfully, but these errors were encountered: