diff options
| author | Elia Pinto <gitter.spiros@gmail.com> | 2014-06-06 07:55:58 -0700 |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2014-06-09 15:53:41 -0700 |
| commit | d0b30a3d4d486040a8a72f60e00b51fe1a3c16ba (patch) | |
| tree | bf5cff02f2ad8d53099b49d5743fe88bf5fd0af1 | |
| parent | 4399345d5e491b6c432f6b3b12aabca5609d19ce (diff) | |
| download | git-d0b30a3d4d486040a8a72f60e00b51fe1a3c16ba.tar.gz | |
t/t5000-tar-tree.sh: avoid "test <cond> -a/-o <cond>"
The construct is error-prone; "test" being built-in in most modern
shells, the reason to avoid "test <cond> && test <cond>" spawning
one extra process by using a single "test <cond> -a <cond>" no
longer exists.
Signed-off-by: Elia Pinto <gitter.spiros@gmail.com>
Reviewed-by: Matthieu Moy <Matthieu.Moy@imag.fr>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
| -rwxr-xr-x | t/t5000-tar-tree.sh | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/t/t5000-tar-tree.sh b/t/t5000-tar-tree.sh index 74fc5a88ec..ad6fa0d4e4 100755 --- a/t/t5000-tar-tree.sh +++ b/t/t5000-tar-tree.sh @@ -72,7 +72,7 @@ check_tar() { for header in *.paxheader do data=${header%.paxheader}.data && - if test -h $data -o -e $data + if test -h $data || test -e $data then path=$(get_pax_header $header path) && if test -n "$path" |