-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
{libllvm,isl,libiberty,libmpc}: fix references to /usr/bin/uname on native FreeBSD #356763
base: staging
Are you sure you want to change the base?
Conversation
de145a9
to
af3ca46
Compare
Seems okay to me, but maybe it's wiser to add |
That would be really excellent! I was told explicitly to not do that when I submitted my first draft of the FreeBSD stdenv rewrite :) |
@@ -14,6 +14,12 @@ stdenv.mkDerivation { | |||
|
|||
postUnpack = "sourceRoot=\${sourceRoot}/libiberty"; | |||
|
|||
# needed until config scripts are updated to not use /usr/bin/uname on FreeBSD native | |||
# updateAutotoolsGnuConfigScriptsHook doesn't seem to work here |
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.
updateAutotoolsGnuConfigScriptsHook only looks in .
, so instead of using sourceRoot
to change directory after unpackPhase
we would need to cd
in preConfigure
to make it work…
Alternatively we could make the directory configurable in the setup hook 🤔
Interesting, what was the reasoning given? FTR, #237968 added it for Linux. I think the inclusion in the Linux stdenv also actively reduces the amount of packages that explicitly request it. I also don't necessarily like running the hook by default, but given that we do it for Linux already… |
The logic is that it will be impossible to remove later. |
af3ca46
to
05dc4b5
Compare
We've all heard this story before. Old versions of gnu config scripts hardcode the /usr/bin/uname filename in the FreeBSD build system code path. Upgrade them so this stops happening. See #313496 for context.
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.