-
Notifications
You must be signed in to change notification settings - Fork 99
setup.py
is replaced by pyproject.toml
#158
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
setup.py
is replaced by pyproject.toml
#158
Conversation
@@ -1 +0,0 @@ | |||
__import__("setuptools").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.
Do not delete this file
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.
Why not? Given the title of the PR?
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.
The move to pyproject.toml was done in #154, this is the correct setup
e4802a7
to
72dd6e1
Compare
Please allow me to rephrase... In what scenarios would the presence of this file be useful? If it was deleted, what would break? |
There are situations and versions of setuptools which require setup.py to be present as a shim for the pyproject.toml, editable installations was one example that can require it. Support for editable installs (as an example) has been added in relatively new versions, but it's a very small burden to support older versions, especially since newer setuptools have their own issues (stricter version specifier parsing, distutils changes, msvc changes are three I've hit this year). So tl;dr to maximize our setuptools compatibility and avoid requiring a relatively new setuptools, the setup.py shim should stay. |
setuptools v61.0.0 (released in March 2022) added support for |
Setuptools v64.0.0 (July 2022) was the first that allowed editable installs without a |
https://packaging.python.org/en/latest/discussions/setup-py-deprecated
In
README.rst
, setup is a noun while set up is a verb.In the removed
setup.py
...https://docs.python.org/3/library/functions.html#import__