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
nbd: add which to buildInputs #46635
Conversation
@@ -11,7 +11,7 @@ stdenv.mkDerivation rec { | |||
patches = [ ./dont-run-make-in-broken-systemd-subdir.patch ]; | |||
|
|||
buildInputs = | |||
[ pkgconfig glib ] | |||
[ pkgconfig glib which ] |
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.
Can you add this to nativeBuildInputs
instead? Also pkgconfig
belongs there.
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.
Gah! One day, I'll get something right the first time.
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 can also cargo-cult this knowledge for examples like pkgconfig
.
A rule of thumb is: Does the system (i.e x86_64) that builds the packages would like to execute this tool during its build and therefore needs the executable for its own cpu architecture or is it a dependencies that is needed for the target platform (aarch64), that the package is build for.
@GrahamcOfBorg build nbd |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: nbd Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: nbd Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: nbd Partial log (click to expand)
|
fd2650d
to
9dc4ab0
Compare
9dc4ab0
to
36bca31
Compare
(No idea what happened with the conflict there. I've rebased, looks ok now.) |
@GrahamcOfBorg build nbd |
sorry, not everyone can trigger builds |
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: nbd Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: nbd Partial log (click to expand)
|
Ah, hehe. Thought I'd give the magic incantation a try :) |
Success on aarch64-linux (full log) Attempted: nbd Partial log (click to expand)
|
Thanks @xeji! |
Motivation for this change
The
nbd
build complains numerously about not being able to find thewhich
package during its testing phase:and so on.
I'm not sure why the build doesn't fail under these conditions, but anyway, simply adding
which
to itsbuildInputs
fixes this.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)