binaryTarball.x86_64-linux fails at /build/install-darwin-multi-user

[FRidh]

nix-build -A binaryTarball.x86_64-linux release.nix -I nixpkgs=../nixpkgs

gives

finalPhase
build time elapsed:  0m0.316s 0m0.302s 13m42.677s 1m27.937s
building '/nix/store/g220pypa90rrkwjis7ir3lp2wk1d7nzk-nix-binary-tarball-1.12pre1234_abcdef.drv'...

In /build/install-darwin-multi-user line 156:
    printf "$BOLD%s$ESC:\t%s\n" "$1" "$2"
                        ^-- SC1117: Backslash is literal in "\t". Prefer explicit escaping: "\\t".
                            ^-- SC1117: Backslash is literal in "\n". Prefer explicit escaping: "\\n".


In /build/install-darwin-multi-user line 221:
    printf "    $ sudo %s\n" "$cmd"
                         ^-- SC1117: Backslash is literal in "\n". Prefer explicit escaping: "\\n".

builder for '/nix/store/g220pypa90rrkwjis7ir3lp2wk1d7nzk-nix-binary-tarball-1.12pre1234_abcdef.drv' failed with exit code 1
error: build of '/nix/store/g220pypa90rrkwjis7ir3lp2wk1d7nzk-nix-binary-tarball-1.12pre1234_abcdef.drv' failed

where Nixpkgs @ 502ebb0 and Nix @ ca40fbd

[orivej]

Fixed in NixOS/nixpkgs#32857

[orivej]

Sorry, that's a different issue.

[edolstra]

Maybe SC1117 should be disabled in release.nix, because that check seems kind of bogus. AFAIK there is nothing wrong with doing printf "\n".

[FRidh]

According to https://github.com/koalaman/shellcheck/wiki/SC1117 its a matter of good practice. In this case, its about being explicit that \n needs to be passed on.