From: Alejandro Colomar <alx@kernel.org>
Date: Fri, 10 Mar 2023 12:21:12 +0000 (+0100)
Subject: lint-man.mk: lint-man-groff-eqn: Fix error detection
X-Git-Tag: man-pages-6.04~60
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0440d04f8317;p=thirdparty%2Fman-pages.git

lint-man.mk: lint-man-groff-eqn: Fix error detection

eqn(1) could theoretically write _only_ newlines to standard error.
That's unlikely, but I'm still worried that someone (even me) might copy
this trick around, and use it in situations where that might actually
happen.  Let's be more precise, and fail when there's literally anything
on standard error.

Reported-by: Ralph Corderoy <ralph@inputplus.co.uk>
Signed-off-by: Alejandro Colomar <alx@kernel.org>
---

diff --git a/lib/lint-man.mk b/lib/lint-man.mk
index 0583964b4c..d47aadc718 100644
--- a/lib/lint-man.mk
+++ b/lib/lint-man.mk
@@ -96,7 +96,7 @@ $(_LINT_man_groff_eqn): %.eqn: %.tbl | $$(@D)/.
 $(_LINT_man_groff_troff): %.troff: %.eqn | $$(@D)/.
 	$(info LINT (eqn)	$@)
 	$(EQN) $(EQNFLAGS) <$< 2>&1 >$@ \
-	| ( ! $(GREP) . )
+	| ( ! $(GREP) ^ )
 
 $(_LINT_man_groff_grotty): %.grotty: %.troff | $$(@D)/.
 	$(info LINT (troff)	$@)