From 9d6c34718b8a43ebc9b48d8a31dcb32a134ca781 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Mon, 17 Jun 2019 18:19:03 +0200 Subject: [PATCH] Produce more verbose error if cppi not found It's fairly easy (especially for new contributors) to not spot the 'cppi not installed' line in the syntax-check output. Add a banner that is shown at the end of syntax-check output if no cppi was found. Signed-off-by: Michal Privoznik ACKed-by: Andrea Bolognani --- cfg.mk | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/cfg.mk b/cfg.mk index 5074ef611a..c0c240b2c0 100644 --- a/cfg.mk +++ b/cfg.mk @@ -1145,6 +1145,11 @@ ifneq ($(_gl-Makefile),) syntax-check: spacing-check test-wrap-argv \ prohibit-duplicate-header mock-noinline group-qemu-caps \ header-ifdef + @if ! cppi --version >/dev/null 2>&1; then \ + echo "*****************************************************" >&2; \ + echo "* cppi not installed, some checks have been skipped *" >&2; \ + echo "*****************************************************" >&2; \ + fi endif # Don't include duplicate header in the source (either *.c or *.h) -- 2.39.5