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
pythonPackages.setuptools: Remove windows files and make reproducible #71101
Conversation
These are various *.exe and *.xml files used only on windows. setuptools includes them by default since it normally creates a single release for all operating systems. This reduces the size from 1020.0K to 801.6K according to `nix-path -sh`.
The egg is a zip file. setuptools leaves timestamps in the egg, which makes the build unreproducible. Unfortunately the files aren't compressed so the size of setuptools increases to 2.3M from 0.8M according to `nix path-info -sh`. With this change, setuptools is reproducible according to nix-build -A python37Packages.setuptools --check
Thank you! I've pushed 90be4c2 instead to reuse the code that is in the hooks. |
90be4c2 broke the
|
This likely has to do with twisted being called in
|
This is indeed the cause of the regression. Thanks for bisecting! Note I did not push this commit, but pushed 90be4c2 instead. It uses hooks instead of the phases defined here, however, it eventually runs the same commands. The next question is, what part exactly is causing it. So, it is not the environment variable |
Fix in #71792. |
This breaks webtest, see #71824 |
Motivation for this change
These are 2 separate commits, but I'm making one pull request against staging to limit rebuilds. Let me know if you would rather split it apart.
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)Notify maintainers
cc @FRidh