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
bat-extras: fix build on Darwin #122069
bat-extras: fix build on Darwin #122069
Conversation
Result of 5 packages built successfully:
Result of 5 packages built successfully:
|
@@ -48,6 +49,7 @@ let | |||
|
|||
# Run the library tests as they don't have external dependencies | |||
doCheck = true; | |||
checkInputs = [ getconf ]; |
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.
This should be behind lib.optionals.
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.
You mean that we just set it on Darwin? I though to keep it always as it is definitively required on both, however somehow included in linux stdenv
or somewhat else ...
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.
I think it is included in glibc for linux. Darwin uses clang so it is not included there. Maybe we want to build it with gccStdenv instead. If not please put it behind stdenv.isDarwin just to be sure.
introduced by NixOS#119689 Signed-off-by: Andreas Schmid <service@aaschmid.de>
0535842
to
9f050c6
Compare
Motivation for this change
fix build failure (#119987) on Darwin introduced by #119689
Things done
sandbox
innix.conf
on non-NixOS linux)nix-build -I nixpkgs=$(pwd) --no-out-link -A bat-extras.batgrep '<nixpkgs>'
./result/bin/
)nix path-info -S
before and after)