-
Notifications
You must be signed in to change notification settings - Fork 10
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
Support setup.cfg #60
Comments
I have only played with standalone From my understanding, in a setting that has
Is it not the case that you can do something similar with a setting if you have |
We don't have setup.py anymore, just pyproject.toml for build-system and tools and setup.cfg for project metadata and requirements. Thanks for the dynamic version trick, I didn't know about that! This request may also become obsolete when we can move to have everything in pyproject.toml. Although then still, it would be great being able to point In the declarative formats (setup.cfg, and maybe in future pyproject.toml), the command class can not be dynamically returned by a getter function like cmdclass =
build_py = mypackage.some_module.build_py |
Setup.py has been replaced by setup.cfg and is now moving towards pyproject.toml. Probably there aren't many projects on setup.py anymore.
While I was cleaning up an old setup.py & miniver project, migration seemed straight forward (with pypa/setuptools#2570 now implemented).
Except that
cmdclass
dict values must be module paths to a public, top-level class. Since the change 1c9f8ca, the build command classes can only be created through function calls, which does not work for the declarative setup.cfg.I tried working around by creating a helper object with lazy properties (so I can preserve the lazy import of super classes), but that is not detected by setuptools and gives an error.
_version.py
…setup.cfg
…In short, first I think it would be good to support setup.cfg, and second, I am not sure whether it is feasible without moving the build command classes to the top level.
The text was updated successfully, but these errors were encountered: