General problem with toolchain.bzl

[SirVer]

Not sure if that is in the modifications you made, but I run into trouble with this line in toolchain.bzl:

["if [[ -v TMPDIR ]]; then mkdir -p $TMPDIR; fi;"] +

With it in, nothing builds on mac os due some shell snafu. Commenting it out makes the build work for me:

$ bazel build //cargo/vendor:pbr
INFO: Analysed target //cargo/vendor:pbr (6 packages loaded).
INFO: Found 1 target...
ERROR: /Users/hrapp/Desktop/Programming/rust/point_cloud_viewer/cargo/vendor/winapi-0.2.8/BUILD:16:1: error executing shell command: 'set -e; if [[ -v TMPDIR ]]; then mkdir -p $TMPDIR; fi; rm -rf bazel-out/darwin-fastbuild/bin/cargo/vendor/winapi-0.2.8/winapi.deps; mkdir bazel-out/darwin-fastbuild/bin/cargo/vendor/winapi-0.2.8/wi...' failed (Exit 2)
/bin/bash: -c: line 0: conditional binary operator expected
/bin/bash: -c: line 0: syntax error near `TMPDIR'
/bin/bash: -c: line 0: `set -e; if [[ -v TMPDIR ]]; then mkdir -p $TMPDIR; fi; rm -rf bazel-out/darwin-fastbuild/bin/cargo/vendor/winapi-0.2.8/winapi.deps; mkdir bazel-out/darwin-fastbuild/bin/cargo/vendor/winapi-0.2.8/winapi.deps'
Target //cargo/vendor/pbr-1.0.0:pbr failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.250s, Critical Path: 0.10s
FAILED: Build did NOT complete successfully

[acmcarther]

Can you use "--verbose_failures" (scrubbing out anything you need to).

That doesn't look like a change I made. Are you on an interesting platform at all? Weird version of bash? Will look into.

[SirVer]

I forgot: this was OS X and I was running a zsh.

…

Am 13.02.2018 um 19:08 schrieb Alex McArther ***@***.***>: Can you use "--verbose_failures" (scrubbing out anything you need to). That doesn't look like a change I made. Are you on an interesting platform at all? Weird version of bash? Will look into. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

[mfarrugi]

@acmcarther this was a part of my dylib changes, iirc it resolved a sandboxing issue in my work environment. I suspect it's not the most portable bash.

…

On Tue, Feb 13, 2018, 13:18 Holger Rapp ***@***.***> wrote: I forgot: this was OS X and I was running a zsh. > Am 13.02.2018 um 19:08 schrieb Alex McArther ***@***.***>: > > Can you use "--verbose_failures" (scrubbing out anything you need to). > > That doesn't look like a change I made. Are you on an interesting platform at all? Weird version of bash? Will look into. > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub, or mute the thread. > — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#31 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACLjePNFCtPVICaXgwP8R-HTqogwCWI5ks5tUdIBgaJpZM4SEK8n> .

[acmcarther]

The PR is here: bazelbuild/rules_rust#61, and I suggested a fix.

[mfarrugi]

That fix went into the PR (though it remains unpushed), that should close this thread.