Code

t3901: avoid negation on right hand side of '|'
authorBrandon Casey <casey@nrlssc.navy.mil>
Mon, 18 May 2009 23:44:42 +0000 (18:44 -0500)
committerJunio C Hamano <gitster@pobox.com>
Tue, 19 May 2009 03:53:16 +0000 (20:53 -0700)
Some shells do not properly handle constructs of the form:

   spew_something | ! process_input

So rewrite this to be:

   spew_something | process_input; test $? != 0

Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t3901-i18n-patch.sh

index 7655da3f8d5e68f293ae5afe2d58dd41b1396f37..b04f74aa719036d96284f8946e163357ab8ba577 100755 (executable)
@@ -19,7 +19,7 @@ check_encoding () {
                8859)
                        grep "^encoding ISO-8859-1" ;;
                *)
-                       ! grep "^encoding ISO-8859-1" ;;
+                       grep "^encoding ISO-8859-1"; test "$?" != 0 ;;
                esac || {
                        bad=1
                        break