[PATCH] tools/nolibc: hide headers_check command by default

Thomas Weißschuh posted 1 patch 3 months, 2 weeks ago
tools/include/nolibc/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] tools/nolibc: hide headers_check command by default
Posted by Thomas Weißschuh 3 months, 2 weeks ago
If V=1 is not specified the executed commands should not be printed.

Hide the commands by default.

Fixes: a6a054c8ad32 ("tools/nolibc: add target to check header usability")
Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>
---
 tools/include/nolibc/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/include/nolibc/Makefile b/tools/include/nolibc/Makefile
index 4bace7737b2ade388e138474c10f027a71e21816..f2f2924626d37ac7cf14f94ec248414fa509dc70 100644
--- a/tools/include/nolibc/Makefile
+++ b/tools/include/nolibc/Makefile
@@ -112,7 +112,7 @@ headers_standalone: headers
 CLANG_CROSS_FLAGS := $(subst --target=s390-linux,--target=systemz-linux,$(CLANG_CROSS_FLAGS))
 
 headers_check: headers_standalone
-	for header in $(filter-out crt.h std.h,$(all_files)); do \
+	$(Q)for header in $(filter-out crt.h std.h,$(all_files)); do \
 		$(CC) $(CLANG_CROSS_FLAGS) -Wall -Werror -nostdinc -fsyntax-only -x c /dev/null \
 			-I$(or $(objtree),$(srctree))/usr/include -include $$header -include $$header || exit 1; \
 	done

---
base-commit: eb135311083100b6590a7545618cd9760d896a86
change-id: 20250623-nolibc-headers-silent-88e4fa592812

Best regards,
-- 
Thomas Weißschuh <linux@weissschuh.net>

Re: [PATCH] tools/nolibc: hide headers_check command by default
Posted by Willy Tarreau 3 months, 2 weeks ago
On Mon, Jun 23, 2025 at 11:29:34PM +0200, Thomas Weißschuh wrote:
> If V=1 is not specified the executed commands should not be printed.
> 
> Hide the commands by default.
> 
> Fixes: a6a054c8ad32 ("tools/nolibc: add target to check header usability")
> Signed-off-by: Thomas Weißschuh <linux@weissschuh.net>


Acked-by: Willy Tarreau <w@1wt.eu>
Willy