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

pypi2nix: add python3packages.setuptool buildinput #24117

Closed
wants to merge 1 commit into from

Conversation

nlewo
Copy link
Member

@nlewo nlewo commented Mar 20, 2017

Fix #24111

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 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.

@mention-bot
Copy link

@nlewo, thanks for your PR! By analyzing the history of the files in this pull request, we identified @garbas to be a potential reviewer.

@nlewo nlewo changed the title pypi2nix: add python3packages.setuptool buildinput wip: pypi2nix: add python3packages.setuptool buildinput Mar 20, 2017
@nlewo nlewo changed the title wip: pypi2nix: add python3packages.setuptool buildinput [wip] pypi2nix: add python3packages.setuptool buildinput Mar 20, 2017
@Mic92
Copy link
Member

Mic92 commented Mar 20, 2017

is this also broken in stable packages?

@Mic92
Copy link
Member

Mic92 commented Mar 20, 2017

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 ];
Copy link
Member

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.

Copy link
Member Author

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.

@nlewo nlewo changed the title [wip] pypi2nix: add python3packages.setuptool buildinput pypi2nix: add python3packages.setuptool buildinput Mar 20, 2017
@nlewo
Copy link
Member Author

nlewo commented Mar 20, 2017

@Mic92 because I was trying to not use python3packages and to understand why it no longer works (point still not completed...).

@FRidh
Copy link
Member

FRidh commented Mar 20, 2017

Note that it looks like @garbas prefers to fetch pyp2nix's dependencies instead of using the already available packages.

@RonnyPfannschmidt
Copy link
Contributor

RonnyPfannschmidt commented Mar 20, 2017

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

@garbas
Copy link
Member

garbas commented Mar 20, 2017

@nlewo can you please open a PR here: https://github.com/garbas/pypi2nix

@garbas garbas closed this Mar 20, 2017
@nlewo
Copy link
Member Author

nlewo commented Mar 20, 2017

@garbas hmm... I'm not able to reproduce when I build it from the repo.

@garbas
Copy link
Member

garbas commented Mar 20, 2017

@nlewo yes, most likely master already has latest setuptools.

@nlewo
Copy link
Member Author

nlewo commented Mar 20, 2017

@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).
I will follow the pipy2nix update in the next weeks. For now, enjoy these moments;)

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

6 participants