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
Fix package distribution #368
Conversation
setup.py
Outdated
"requests[security] >= 2.0.0", | ||
], | ||
':python_version>="3.3"': [ | ||
"requests >= 2.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't requests
a required package for any version of python?
Also we seem to have lost the cryptography
package on < 3.0, idna
package on < 3.0, pyOpenSSL
on < 3.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those come with requests[security]
. That's why I split it up, because Python 2.7 needs the extra installation of requests
, but Python >= 3.3 doesn't.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
does requests[security]
also install the core requests
package?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also can we make this python_version<3 instead of ==2.7 so that someone on a unsupported version can at least try to install it :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto for python_version>=3.3 => 3.0
This PR fixes package distribution, so
twilio-python
installs the correct dependencies based on the Python version it will be running on.