Mon Nov 28 23:15:34 2022 UTC ()
check-files.mk: fix execution when CHECK_FILES_STRICT=yes

Commit r. 1.2044 "mk: Don't define DO_NADA to true." of bsd.pkg.mk
caused the checks executed when CHECK_FILES_STRICT=yes is set to fail
(e.g., "/bin/sh: -c: line 1: syntax error near unexpected token `;'").

(The pre-existing coding style that uses "true;" rather than a bare ":"
instead has been retained, in case there is an obscure (?) reason for
that approach.)


(gutteridge)
diff -r1.38 -r1.39 pkgsrc/mk/check/check-files.mk
cvs diff -r1.38 -r1.39 pkgsrc/mk/check/check-files.mk (expand / switch to context diff)
--- pkgsrc/mk/check/check-files.mk 2022/11/23 11:55:43 1.38
+++ pkgsrc/mk/check/check-files.mk 2022/11/28 23:15:34 1.39
@@ -1,4 +1,4 @@
-# $NetBSD: check-files.mk,v 1.38 2022/11/23 11:55:43 jperkin Exp $
+# $NetBSD: check-files.mk,v 1.39 2022/11/28 23:15:34 gutteridge Exp $
 #
 # This file checks that the list of installed files matches the PLIST.
 # For that purpose it records the file list of LOCALBASE before and
@@ -340,7 +340,7 @@
 	${RUN}					\
 	if ${CMP} -s ${_CHECK_FILES_PRE.sysconfdir}			\
 		     ${_CHECK_FILES_POST.sysconfdir}; then		\
-		${DO_NADA};						\
+		${TRUE};						\
 	else								\
 		${ECHO} "************************************************************"; \
 		${ECHO} "The package has modified ${PKG_SYSCONFDIR}"	\
@@ -360,7 +360,7 @@
 	${RUN}					\
 	if ${CMP} -s ${_CHECK_FILES_PRE.varbase}			\
 		       ${_CHECK_FILES_POST.varbase}; then		\
-		${DO_NADA};						\
+		${TRUE};						\
 	else								\
 		${ECHO} "************************************************************"; \
 		${ECHO} "The package has modified ${VARBASE}"		\