diff options
author | Jeff King <peff@peff.net> | 2015-03-25 01:29:10 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2015-03-25 10:24:13 -0700 |
commit | 76e057dba29af1f322a19cb75b2921010dbe459c (patch) | |
tree | 366fda4723074b4707c7ffa77a1b90c72edcc261 /t/t7701-repack-unpack-unreachable.sh | |
parent | t3305: fix ignored exit code inside loop (diff) | |
download | tgif-76e057dba29af1f322a19cb75b2921010dbe459c.tar.xz |
t7701: fix ignored exit code inside loop
When checking a list of file mtimes, we use a loop and break
out early from the loop if any entry does not match.
However, the exit code of a loop exited via break is always
0, meaning that the test will fail to notice we had a
mismatch. Since the loop is inside a function, we can fix
this by doing an early "return 1".
Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 't/t7701-repack-unpack-unreachable.sh')
-rwxr-xr-x | t/t7701-repack-unpack-unreachable.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/t/t7701-repack-unpack-unreachable.sh b/t/t7701-repack-unpack-unreachable.sh index aad8a9c64d..b66e383866 100755 --- a/t/t7701-repack-unpack-unreachable.sh +++ b/t/t7701-repack-unpack-unreachable.sh @@ -57,7 +57,7 @@ compare_mtimes () { read tref rest && while read t rest; do - test "$tref" = "$t" || break + test "$tref" = "$t" || return 1 done } |