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
hidapi: drop cross conditional on gnum4 #108044
Conversation
f651423 only added this in the cross case to avoid another mass rebuild, but there's no reason to not clean this up during the next staging cycle.
6b85c43
to
47e1583
Compare
This also includes the review feedback from @SuperSandro2000 in #108025. |
Just an idea - but shouldn't we just add |
This sounds right - I wonder why it was available in non-cross builds in first place, though? |
On the another hand, it seems there are only 4 derivations requiring both I am wondering how many cases are we missing, though ... |
|
note the m4 alias is more commonly used. |
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.
We need to fix propagation of m4 and probably remove gnum4 here as dependency. #108047
I am wrong here. PATH is good. HOST_PATH is where it should not 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.
diff LGTM
So the issue was bf46afd...doh |
PR with the style changes is of course still welcome! |
f651423 only added this in the cross
case to avoid another mass rebuild, but there's no reason to not clean
this up during the next staging cycle.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)