Merge pull request #52679 from ixdy/bazel-comprehension-variables

Automatic merge from submit-queue (batch tested with PRs 52679, 52285). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.. bazel: set --incompatible_comprehension_variables_do_not_leak=false **What this PR does / why we need it**: future-proofing the repo against the upcoming release of bazel 0.6.0. x-ref #52677 **Release note**: ```release-note NONE ``` /assign @spxtr @BenTheElder @mikedanese

Merge pull request #52679 from ixdy/bazel-comprehension-variables
4fea2136 · Kubernetes Submit Queue · GitHub · a8b3d38b · 59906e29 · 4fea2136
Commit 4fea2136 authored Sep 19, 2017 by Kubernetes Submit Queue Committed by GitHub Sep 19, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

.bazelrc build/root/.bazelrc +6 -0

No files found.
--- a/build/root/.bazelrc
+++ b/build/root/.bazelrc
@@ -11,3 +11,9 @@ build --sandbox_tmpfs_path=/tmp
 # Ensure that Bazel never runs as root, which can cause unit tests to fail.
 # This flag requires Bazel 0.5.0+
 build --sandbox_fake_username
+
+# rules_go@82483596ec203eb9c1849937636f4cbed83733eb has a typo that
+# inadvertently relies on comprehension variables leaking.
+# TODO(ixdy): Remove this default once rules_go is bumped.
+# Ref kubernetes/kubernetes#52677
+build --incompatible_comprehension_variables_do_not_leak=false