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
{cc,bintools}-wrapper: Some fixes #92089
{cc,bintools}-wrapper: Some fixes #92089
Conversation
This means we can freely keep the comments up to date without the penalty of a mass rebuild.
This will unbreak firefox and a few other packages which try to grab some of the libcxx flags.
@@ -228,58 +231,56 @@ stdenv.mkDerivation { | |||
*) echo "Multiple dynamic linkers found for platform '${targetPlatform.config}'." >&2;; | |||
esac | |||
|
|||
if [ -n "''${dynamicLinker:-}" ]; then | |||
if [ -n "''${dynamicLinker-}" ]; then |
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.
I'm mainly curious... this dash is some kind of special operator? I can't find this one in man bash
.
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.
@vcunat See https://wiki.bash-hackers.org/syntax/pe#use_a_default_value
If you omit the
:
(colon), like shown in the second form, the default value is only used when the parameter was unset, not when it was empty.
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.
Thanks. Now I also read that man bash
part again – more properly – and I see it there as well 🤦
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.
Let's have this in the current staging-next
iteration. The risks seem low enough and otherwise we'd have to work around those problems in some other way.
Motivation for this change
This means we can freely keep the comments up to date without the penalty of a mass rebuild.
This will unbreak firefox and a few other packages which try to grab some of the libcxx flags.
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)