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
buildPythonPackage: add support for setupPyGlobalFlags (2) #64701
Conversation
I think this is a good time to also rename the two options to get rid of the |
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.
Small moment: the commit message is still about setupPyDistFlags
;)
Didn't actually test it now because I'll die rebuilding all the world and Tensorflow. Testing it on virt-manager
would be easier. @aske?
BTW we'll also need to test pyproject-using packages with flags - maybe they actually needed global flags all along. |
There are not that many upstreams that already use |
I would like to rename the options to |
Sounds good to me! We could leave old options and append them to the new ones with |
Some bike-shedding: |
I'd say |
Anyway, |
I will keep the renaming for another time. |
Motivation for this change
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)