diff options
| author |  Damien Martin-Guillerez <dmarting@google.com> | 2015-07-02 08:38:58 +0000 |
|---|---|---|
| committer |  Florian Weikert <fwe@google.com> | 2015-07-02 09:23:34 +0000 |
| commit | 4f89281d96cf039a483a935074124bf5f42506bb (patch) | |
| tree | f54a74de0874c60ad9c0f18cbd1a7b7459eb5252 | |
| parent | 3ebad611865c90ebec047a773ac07c769d2b2af5 (diff) | |
Fixed the deterministic test flakiness
If you already ran the deterministic test with an earlier version of
the depot, the second run would build only once Bazel and use the
previous run log to do the diff, leading to an obvious failure as
the code did change. It was due to an incorrect parameter default
value that pointed to the diff of the previous run, skipping the
first bazel compilation.
--
MOS_MIGRATED_REVID=97399620
| -rwxr-xr-x | compile.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/compile.sh b/compile.sh index 9523edf1bb..da1667abc7 100755 --- a/compile.sh +++ b/compile.sh @@ -59,7 +59,7 @@ function parse_options() { [[ "${COMMANDS}" =~ (test|all) ]] && DO_TESTS=1 BAZEL_BIN=${2:-"bazel-bin/src/bazel"} - BAZEL_SUM=${3:-"bazel-out/bazel_checksum"} + BAZEL_SUM=${3:-"x"} } parse_options "${@}" |