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
wfuzz: init at 2.4.2 #76058
wfuzz: init at 2.4.2 #76058
Conversation
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.
If want this available as a python package and as an application. I would put the expression in python-modules, and then use with python3Packages; toPythonApplication wfuzz;
the problem right now is that you are introducing the python37 interpreter to all version of the python modules currently
Changed. I don't quite understand what the problem is though... |
your original expression had That means for python2, it was actually building and propagating the python37 interpreter for the python2 package |
to have it your original way, you would need to override the interpreter for each python module, similiar to nixpkgs/pkgs/top-level/python-packages.nix Line 254 in 1c15955
|
Thanks for the explanation! |
1267c2d
to
47c2257
Compare
b4dd7f4
to
a125ecb
Compare
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.
diff LGTM
commit LGTM
has tests 👍
executable shows usage
[6 built, 15 copied (31.0 MiB), 5.2 MiB DL]
https://github.com/NixOS/nixpkgs/pull/76058
3 package were built:
python27Packages.wfuzz wfuzz python38Packages.wfuzz
@GrahamcOfBorg build python27Packages.wfuzz wfuzz python38Packages.wfuzz |
arch build is broken for unrelated reasons |
Motivation for this change
Make wfuzz available on Nix.
Contains the related Python3 package.
Things done
sandbox
innix.conf
on non-NixOS linux)./result/bin/
)nix path-info -S
before and after)