diff options
| author | Jon Seymour <jon.seymour@gmail.com> | 2011-08-04 12:00:58 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2011-08-04 22:33:34 (GMT) |
| commit | 6ba7acffdd24e9ed9b31e503ce87f2af31cd0883 (patch) | |
| tree | a11c1e81edc6caf54102f20fa6fd84a4ba49935e /git-bisect.sh | |
| parent | 4764f46492b0849ce3cf2f527978acd9c2d5f7e6 (diff) | |
| download | git-6ba7acffdd24e9ed9b31e503ce87f2af31cd0883.zip git-6ba7acffdd24e9ed9b31e503ce87f2af31cd0883.tar.gz git-6ba7acffdd24e9ed9b31e503ce87f2af31cd0883.tar.bz2 | |
bisect: use && to connect statements that are deferred with eval.
Christian Couder pointed out that the existing eval strategy
swallows an initial non-zero return. Using && to connect
the statements should fix this.
Signed-off-by: Jon Seymour <jon.seymour@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'git-bisect.sh')
| -rwxr-xr-x | git-bisect.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/git-bisect.sh b/git-bisect.sh index 20f6dd5..a44ffe1 100755 --- a/git-bisect.sh +++ b/git-bisect.sh @@ -86,7 +86,7 @@ bisect_start() { 0) state='bad' ; bad_seen=1 ;; *) state='good' ;; esac - eval="$eval bisect_write '$state' '$rev' 'nolog'; " + eval="$eval bisect_write '$state' '$rev' 'nolog' &&" shift ;; esac @@ -145,7 +145,7 @@ bisect_start() { # echo "$start_head" >"$GIT_DIR/BISECT_START" && git rev-parse --sq-quote "$@" >"$GIT_DIR/BISECT_NAMES" && - eval "$eval" && + eval "$eval true" && echo "git bisect start$orig_args" >>"$GIT_DIR/BISECT_LOG" || exit # # Check if we can proceed to the next bisect state. |