Skip to content
Snippets Groups Projects
Commit 6c7a58d3 authored by Jun Wu's avatar Jun Wu
Browse files

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
parent 48046444
No related branches found
No related tags found
No related merge requests found
......@@ -8,7 +8,7 @@
(The writing "no-che?k-code" is for not skipping this file when checking.)
$ hg locate -X contrib/python-zstandard -X hgext/fsmonitor/pywatchman |
> sed 's-\\-/-g' | xargs "$check_code" --warnings --per-file=0 || false
> sed 's-\\-/-g' | "$check_code" --warnings --per-file=0 - || false
contrib/perf.py:869:
> r.revision(r.node(x))
don't convert rev to node before passing to revision(nodeorrev)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment