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
pypi2nix: add python3packages.setuptool buildinput #24117
Conversation
is this also broken in stable packages? |
Works for me, why is this still work in progress? |
@@ -27,7 +27,7 @@ in stdenv.mkDerivation rec { | |||
click | |||
requests | |||
]; | |||
buildInputs = [ python zip makeWrapper nix.out nix-prefetch-scripts ]; | |||
buildInputs = [ python zip makeWrapper nix.out nix-prefetch-scripts python3Packages.setuptools ]; |
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.
use python.pkgs.setuptools
to guarantee the correct version of setuptools
is used.
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.
@FRidh ok, thanks, and done.
@Mic92 because I was trying to not use python3packages and to understand why it no longer works (point still not completed...). |
Note that it looks like @garbas prefers to fetch pyp2nix's dependencies instead of using the already available packages. |
as far as i understood the reason for having own package-sets is to remove the dependency on the currently changing python ecosystem edit: in nixos |
@nlewo can you please open a PR here: https://github.com/garbas/pypi2nix |
@garbas hmm... I'm not able to reproduce when I build it from the repo. |
@nlewo yes, most likely master already has latest setuptools. |
@garbas I'm really curious to understand what is happening... I tryied to build the nixpkgs packages with the current pypi2nix master but it failed (while it doens't fail when I build it from the pypi2nix repo itself). |
Fix #24111
Motivation for this change
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)