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
yoda: 1.6.5 -> 1.6.6, provide version with ROOT #24583
Conversation
}; | ||
|
||
pythonPath = []; # python wrapper support | ||
|
||
buildInputs = with python2Packages; [ python numpy matplotlib makeWrapper ]; | ||
buildInputs = with python2Packages; [ python numpy matplotlib root makeWrapper ]; |
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.
What is the disadvantage of having this enabled by default?
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.
ROOT adds 433 mb to closure and the only practical reason to enable it is to have root2yoda and yoda2root utilities. The reverse dependencies of YODA don't care about those. So yoda-with-root is for using interactively when user need those extra conversion utilities (most often you don't need those, actually).
|
I guess this is one of those instances when nix-prefetch-url returns something completely different from what nix-build sees. I fixed the hashes. @rasendubi Sorry and thank you! |
|
I did not realize that. Thanks. Also in my tests |
I believe |
@vcunat In my tests I was creating a separate attribute to overcome the list problem. I believe that still didn't get a correct hash even after that. |
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/
)