Skip to content

mkPythonDerivation: obey a dontWrapPythonPrograms attribute #26142

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

Closed
wants to merge 2 commits into from

Conversation

risicle
Copy link
Contributor

@risicle risicle commented May 26, 2017

Motivation for this change

85a87f5 broke python packages not wanting all potential "programs" wrapped with wrapPythonPrograms but also not wanting to have to replace/disable the whole fixupPhase. Notably django.

I propose borrowing a leaf out of mkDerivation's book and making mkPythonDerivation obey a dontWrapPythonPrograms flag.

This includes a followup commit adding the flag to django packages to fix them.

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.

Sorry, something went wrong.

risicle added 2 commits May 25, 2017 20:56
This should allow a python derivation to disable possibly overzealous
python program wrapping without having to override the whole fixupPhase
@risicle
Copy link
Contributor Author

risicle commented May 27, 2017

The two failures appear to be because of log limits exceeding again - it would be nice if some more people could do some local builds of this because I'm having trouble building current python35Packages.django_hijack with-or-without this patch and would like to to make sure it's a totally independent failure.

@FRidh
Copy link
Member

FRidh commented May 27, 2017

@risicle note that, if you want to limit wrapping to only $out/bin, then that will soon be the case since I plan to push aa86d9fa7a93023f6c54f45631505375dc628b91.

@risicle
Copy link
Contributor Author

risicle commented May 27, 2017

Oh man. Perhaps this is best ignored then. It's a potentially useful feature, but adding a feature-with-no-users feels more like noise.

@FRidh
Copy link
Member

FRidh commented May 27, 2017

I've pushed 45ac770 to master because I think it will come to use at some point.

@FRidh FRidh closed this May 27, 2017
@FRidh
Copy link
Member

FRidh commented May 27, 2017

and also c10af9e so we don't need the other commit anymore.

@risicle
Copy link
Contributor Author

risicle commented May 27, 2017

ok fair enough

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants