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
thunderbolt: 0.9.2 -> 0.9.3 #42743
thunderbolt: 0.9.2 -> 0.9.3 #42743
Conversation
Fixed up `cmakeFlags` so `tbtacl`, `tbtacl-write`, `tbtxdomain`, and the udev rules now show up in the derivation output. Previously there was only `tbtadm`.
Success on x86_64-linux (full log) Attempted: thunderbolt Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: thunderbolt Partial log (click to expand)
|
"-DUDEV_BIN_DIR=$out/bin" | ||
"-DUDEV_RULES_DIR=$out/udev" | ||
]; | ||
# These can't go in the normal nix cmakeFlags because $out needs to be |
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.
See also #37693
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.
Is it okay to use placeholder expressions in nixpkgs now?
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.
Unfortunately not 😢
Success on x86_64-linux (full log) Attempted: thunderbolt Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: thunderbolt Partial log (click to expand)
|
Fixed up
cmakeFlags
sotbtacl
,tbtacl-write
,tbtxdomain
, and the udev rules now show up in the derivation output. Previously there was onlytbtadm
.Motivation for this change
New version.
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)