comparison tests/test-check-code.t @ 31825:6c7a58d3ca5d

test-check-code: do not use xargs We have too many files, and passing them via arguments could cause strange errors on some platforms [1]. Since check-code.py can now take "-" and read file names from stdin, use it instead of xargs to avoid the argv size limit. [1]: https://www.mercurial-scm.org/pipermail/mercurial-devel/2017-April/096346.html
author Jun Wu <quark@fb.com>
date Thu, 06 Apr 2017 22:10:46 -0700
parents f6d77af84ef3
children 3e03a4b9ec8c
comparison
equal deleted inserted replaced
31824:4804644489cf 31825:6c7a58d3ca5d
6 6
7 New errors are not allowed. Warnings are strongly discouraged. 7 New errors are not allowed. Warnings are strongly discouraged.
8 (The writing "no-che?k-code" is for not skipping this file when checking.) 8 (The writing "no-che?k-code" is for not skipping this file when checking.)
9 9
10 $ hg locate -X contrib/python-zstandard -X hgext/fsmonitor/pywatchman | 10 $ hg locate -X contrib/python-zstandard -X hgext/fsmonitor/pywatchman |
11 > sed 's-\\-/-g' | xargs "$check_code" --warnings --per-file=0 || false 11 > sed 's-\\-/-g' | "$check_code" --warnings --per-file=0 - || false
12 contrib/perf.py:869: 12 contrib/perf.py:869:
13 > r.revision(r.node(x)) 13 > r.revision(r.node(x))
14 don't convert rev to node before passing to revision(nodeorrev) 14 don't convert rev to node before passing to revision(nodeorrev)
15 Skipping i18n/polib.py it has no-che?k-code (glob) 15 Skipping i18n/polib.py it has no-che?k-code (glob)
16 mercurial/demandimport.py:312: 16 mercurial/demandimport.py:312: