From 00babda45eec2e2ea4274578b4b2f48c48e6eaa7 Mon Sep 17 00:00:00 2001 From: Martin Kletzander Date: Thu, 3 Mar 2022 13:14:18 +0100 Subject: [PATCH] syntax-check: Rework mock-noinline to get all files at once MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script can break if the number of files does not fit one invocation and xargs has to split it. Instead pipe the list of files directly into the script and in the script read them from stdin instead of the arguments. Signed-off-by: Martin Kletzander Reviewed-by: Michal Privoznik Reviewed-by: Ján Tomko --- build-aux/syntax-check.mk | 2 +- scripts/mock-noinline.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/build-aux/syntax-check.mk b/build-aux/syntax-check.mk index 9407581d0e..a8c9153b20 100644 --- a/build-aux/syntax-check.mk +++ b/build-aux/syntax-check.mk @@ -1549,7 +1549,7 @@ sc_spacing-check: { echo '$(ME): incorrect formatting' 1>&2; exit 1; } sc_mock-noinline: - $(AM_V_GEN)$(VC_LIST_EXCEPT) | $(GREP) '\.[ch]$$' | $(RUNUTF8) xargs \ + $(AM_V_GEN)$(VC_LIST_EXCEPT) | $(GREP) '\.[ch]$$' | $(RUNUTF8) \ $(PYTHON) $(top_srcdir)/scripts/mock-noinline.py sc_header-ifdef: diff --git a/scripts/mock-noinline.py b/scripts/mock-noinline.py index 712550cb7a..13c296603d 100644 --- a/scripts/mock-noinline.py +++ b/scripts/mock-noinline.py @@ -63,7 +63,7 @@ def scan_overrides(filename): mocked[name] = "%s:%d" % (filename, lineno) -for filename in sys.argv[1:]: +for filename in sys.stdin.readlines(): if filename.endswith(".h"): scan_annotations(filename) elif filename.endswith("mock.c"):