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

Use extras_require over install_requires in mypy_extensions #5511

Closed
wants to merge 1 commit into from
Closed

Use extras_require over install_requires in mypy_extensions #5511

wants to merge 1 commit into from

Conversation

sersorrel
Copy link

There are still old versions of setuptools hanging around that don't understand environment markers in install_requires (e.g. in Debian Jessie, see common-workflow-language/cwltool#872) and it would be nice if mypy_extensions would be installable on those versions of setuptools, especially since the main mypy setup.py uses extras_require for all its environment markers.

Copy link
Member

@ilevkivskyi ilevkivskyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ethanhs are there any potential downsides/dangers? If not, then I would propose to merge this and make a point release of mypy_extensions.

@emmatyping
Copy link
Collaborator

I don't think there is a downside, but I also don't think it is needed if we release mypy_extensions as a wheel (which is a good thing to do in general). According to Hynek (here), only pip 1.4 is needed to install a wheel which uses install_requires. However, with a source distribution, a rather recent version of setuptools is needed to build the package. I'd prefer to release a wheel, as using extras_require in this way is not really documented.

@ilevkivskyi
Copy link
Member

I'd prefer to release a wheel, as using extras_require in this way is not really documented.

@anowlcalledjosh Does this solution work for for you? It would be easy to just upload wheels to PyPI.

@sersorrel
Copy link
Author

Assuming that:

only pip 1.4 is needed to install a wheel which uses install_requires

then that sounds fine to me - I have no desire to perpetuate the extras_require hack if it's not necessary.

@ilevkivskyi
Copy link
Member

OK, I will make micro-release mypy_extensions 0.4.1 now.

@ilevkivskyi
Copy link
Member

OK, I just uploaded the source and wheels distributions for mypy_extensions 0.4.1.

Closing this for now. Please re-open or create a new issue if you still have problems.

@ilevkivskyi
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants