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
virtualbox: remove useless patchelfUnstable build input #47461
Conversation
It wasn't used at all because stdenv puts regular patchelf in PATH before it.
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: virtualbox Partial log (click to expand)
|
No attempt on aarch64-linux (full log) The following builds were skipped because they don't evaluate on aarch64-linux: virtualbox Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: virtualbox Partial log (click to expand)
|
Ha! Okay sure if my PR is needed to fix this then arguably it never did anything and since it seems to work regardless.... Even so I kinda would like to stop the production of errors like this (copied from Borg output above):
which is what that PR resolves, if a bit messily. Honestly I'm good with this if we don't like the approach/fix proposed there, but in absence of pushback prefer avoiding the errors and aborted processes. |
But maybe that motivates just using patchelfUnstable by default instead. :) |
For completeness and reference following ease, the PR I refer to above is this one: #47444 Sorry for not linking in the first place :). |
Let's get rid of these weird errors. Closing in favor of #47444. |
Motivation for this change
It wasn't used at all because stdenv puts regular
patchelf
inPATH
before it, see discussion in #45449Things done
cc @dtzWill