diff options
author | Richard Hansen <rhansen@bbn.com> | 2013-11-18 05:12:47 +0100 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2013-11-18 22:45:58 +0100 |
commit | 25607db2c30c618a18f66350f01b1ccb05c551c6 (patch) | |
tree | f2c517b60d60b9e292b04f796efdc0226803933c /contrib/remote-helpers/test-hg.sh | |
parent | test-hg.sh: eliminate 'local' bashism (diff) | |
download | git-25607db2c30c618a18f66350f01b1ccb05c551c6.tar.xz git-25607db2c30c618a18f66350f01b1ccb05c551c6.zip |
test-hg.sh: avoid obsolete 'test' syntax
The POSIX spec says that the '-a', '-o', and parentheses operands to
the 'test' utility are obsolete extensions due to the potential for
ambiguity. Replace '-o' with '|| test' to avoid unspecified behavior.
Signed-off-by: Richard Hansen <rhansen@bbn.com>
Reviewed-by: Felipe Contreras <felipe.contreras@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to '')
-rwxr-xr-x | contrib/remote-helpers/test-hg.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/contrib/remote-helpers/test-hg.sh b/contrib/remote-helpers/test-hg.sh index ceef6b1917..5535e8c9be 100755 --- a/contrib/remote-helpers/test-hg.sh +++ b/contrib/remote-helpers/test-hg.sh @@ -83,7 +83,7 @@ check_push () { test $ref_ret -ne 0 && echo "match for '$branch' failed" && break done - if test $expected_ret -ne $ret -o $ref_ret -ne 0 + if test $expected_ret -ne $ret || test $ref_ret -ne 0 then return 1 fi |