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
make-system-tarball.sh: double quote to prevent globbing and word splitting. #49427
Conversation
SC2086: Double quote to prevent globbing and word splitting.
SC2086: Double quote to prevent globbing and word splitting.
fi | ||
done | ||
|
||
$extraCommands | ||
"$extraCommands" |
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.
The others look good but I think this one needs to be left unquoted. For instance this script fails in bash:
> extraCommands='echo 1'
> "$extraCommands"
test.sh: line 3: echo 1: command not found
Because it thinks echo 1 is a path name.
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.
Maybe an eval "$extraCommands"
would do?
|
||
sources_=($sources) | ||
targets_=($targets) | ||
|
||
echo $objects | ||
echo "$objects" |
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.
Suggesting to ✂️ this line altogether. This is only for debugging, I believe.
Are there any updates on this pull request, please? |
Thank you for your contributions.
|
I marked this as stale due to inactivity. → More info |
Motivation for this change
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)