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
buildRustCrate: fix equality testing #37928
Conversation
Use string equality instead of integer equality.
cc @P-E-Meunier |
@@ -19,7 +19,7 @@ let makeDeps = dependencies: | |||
echo_build_heading() { | |||
start="" | |||
end="" | |||
if [[ x"${colors}" -eq x"always" ]]; then | |||
if [[ x"${colors}" = x"always" ]]; 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.
Is this x"
stuff needed in any posix shell from this century?
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 have absolutely no idea - I made the minimal changes to make it work. I'm not very good at bash.
Wow, cool, thanks for that! |
Let's leave x"" to the 1990s, where they belong
@GrahamcOfBorg build cargo-vendor |
Success on x86_64-darwin (full log) Attempted: cargo-vendor Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: cargo-vendor Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: cargo-vendor Partial log (click to expand)
|
Use string equality instead of integer equality.
Motivation for this change
Compiling a crate named cargo-watch results in a binary named cargo_watch due to a script error - the logic already existed to name the binary correctly, but used the wrong comparison operator and so failed.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)