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
valgrind: disable on darwin #101442
valgrind: disable on darwin #101442
Conversation
(stdenv.lib.optionalString (!stdenv.isDarwin) valgrind) | ||
libxslt | ||
]; |
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.
(stdenv.lib.optionalString (!stdenv.isDarwin) valgrind) | |
libxslt | |
]; | |
libxslt | |
] ++ stdenv.lib.optionals (!stdenv.isDarwin) [ valgrind ]; |
besides that, LGTM
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.
That is what I did first but it causes unnecessary rebuilds on linux.
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.
oh, interesting, I'm not sure why it should cause rebuilds on Linux
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.
The order of build inputs is preserved into the build process itself.
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.
TIL thanks!
Pushed a cleaned up version of this to staging: d126f0b |
Unfortunately there's another problem: https://hydra.nixos.org/job/nixpkgs/trunk/nix-info-tested.x86_64-darwin |
Our macOS SDKs are from 10.12 but our cctools is from 10.14 so I think that basically kills off
valgrind
on darwin as upstream doesn't support it on >= 10.13 and no one seems to be building it on >= 10.14.The
libpsl
commit should be enough to unblocknixpkgs-unstable
.