-
Notifications
You must be signed in to change notification settings - Fork 1.1k
setup.py
: Remove setup_requires
, remove setuptools from `install_requires'
#2118
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
Conversation
Benchmarks that have stayed the same:
|
Could you elaborate on how setup_requires is outdated and pyproject.toml is read by |
It's true that |
Hey @mkoeppe, I had a few questions that I wasn't understanding about the longer term directions for our setup.py and pyproject.toml; I'm being a bit presumptuous in assuming you're familiar with the python packaging ecosystem, so no worries if you don't have a good answer on what the new best practices are!
|
Hi @PTNobel , what is deprecated is calling In modern Python packaging, The build backend is declared in Declaring metadata such as runtime dependencies (install-requires) in |
Thanks for the explanation! (Also I'm a huge SageMath fan! Thanks for all your work there! I've been using it since I was 15!) I'll have to experiment with the alternative build systems and see if we can find something better for our setup. |
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.
Thanks for the detailed explanation @mkoeppe!
Description
Please include a short summary of the change.
Issue link (if applicable):
setup_requires
is outdated; the build dependencies are declared inpyproject.toml
.Also
setuptools
should not appear ininstall_requires
; these are the runtime dependencies.Type of change
Contribution checklist