Skip to content
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

Python: several fixes (3) #29072

Merged
merged 9 commits into from Sep 7, 2017
Merged

Python: several fixes (3) #29072

merged 9 commits into from Sep 7, 2017

Conversation

FRidh
Copy link
Member

@FRidh FRidh commented Sep 7, 2017

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • Linux
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Fits CONTRIBUTING.md.

…ixes NixOS#25428

While we tell pip not to fetch (with the `--no-index` option),
`setuptools` can do so itself. In the past we used a `distutils.cfg`
with `allow-hosts = None` to prevent setuptools from fetching itself.
This was removed when we started building wheels in
2562f94de4e4fd2ddc677187fa2e2848L69.

The `dist-utils.cfg` code was still there, and adding it to
`buildInputs` is sufficient.

Tested with python.pkgs.passlib by removing the `checkInputs` / `nose`.
@FRidh FRidh changed the title Python fixes 3 Python: several fixes (3) Sep 7, 2017
@FRidh FRidh merged commit 56c56fb into NixOS:master Sep 7, 2017
FRidh added a commit that referenced this pull request Sep 7, 2017
Python: several fixes (3)
(cherry picked from commit 56c56fb)
@FRidh FRidh mentioned this pull request Sep 8, 2017
@FRidh FRidh deleted the python-fixes-3 branch October 20, 2017 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant