[PATCH STABLE] test-check-code-hg: fix xargs exit status on OSX

Patrick Mezard patrick at mezard.eu
Thu Feb 23 10:52:10 CST 2012


# HG changeset patch
# User Patrick Mezard <patrick at mezard.eu>
# Date 1330015617 -3600
# Branch stable
# Node ID e4009152b804009bab8de45c5cfe899a16e1084b
# Parent  616c2e278f18984dc48b80bc56b55da626130709
test-check-code-hg: fix xargs exit status on OSX

When xargs subcommand invocation fails in a normal way, GNU xargs returns 123
and BSD one returns 1.

diff --git a/tests/test-check-code-hg.t b/tests/test-check-code-hg.t
--- a/tests/test-check-code-hg.t
+++ b/tests/test-check-code-hg.t
@@ -3,7 +3,7 @@
 
   $ hg manifest | xargs "$check_code" || echo 'FAILURE IS NOT AN OPTION!!!'
 
-  $ hg manifest | xargs "$check_code" --warnings --nolineno --per-file=0
+  $ hg manifest | xargs "$check_code" --warnings --nolineno --per-file=0 || true
   contrib/check-code.py:0:
    > #    (r'^\s+[^_ \n][^_. \n]+_[^_\n]+\s*=', "don't use underbars in identifiers"),
    warning: line over 80 characters
@@ -657,4 +657,3 @@
   tests/test-walkrepo.py:0:
    >         print "Found %d repositories when I should have found 3" % (len(reposet),)
    warning: line over 80 characters
-  [123]


More information about the Mercurial-devel mailing list