Content-Transfer-Encoding: 7bit
Content-Type: multipart/mixed; boundary="_----------=_1602006482183640"
MIME-Version: 1.0
Date: Tue, 6 Oct 2020 17:48:02 +0000
From: "Roland Illig" <rillig@netbsd.org>
Subject: CVS commit: pkgsrc/mk
To: pkgsrc-changes@NetBSD.org
Reply-To: rillig@netbsd.org
Message-Id: <20201006174802.D29CCFB28@cvs.NetBSD.org>
Sender: pkgsrc-changes-owner@NetBSD.org
Precedence: bulk

This is a multi-part message in MIME format.

--_----------=_1602006482183640
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="US-ASCII"

Module Name:	pkgsrc
Committed By:	rillig
Date:		Tue Oct  6 17:48:02 UTC 2020

Modified Files:
	pkgsrc/mk: subst.mk

Log Message:
mk/subst.mk: remove SUBST_NOOP_OK

This means that from now on, there is no global setting to switch off
this redundancy check.  Individual SUBST classes can still set their own
SUBST_NOOP_OK.<id> in order to ignore no-op filename patterns.

The current bulk builds do not show any build failures that are caused
by this, which means that really almost all packages have been migrated.


To generate a diff of this commit:
cvs rdiff -u -r1.102 -r1.103 pkgsrc/mk/subst.mk

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.


--_----------=_1602006482183640
Content-Disposition: inline
Content-Length: 2517
Content-Transfer-Encoding: binary
Content-Type: text/x-diff; charset=us-ascii

Modified files:

Index: pkgsrc/mk/subst.mk
diff -u pkgsrc/mk/subst.mk:1.102 pkgsrc/mk/subst.mk:1.103
--- pkgsrc/mk/subst.mk:1.102	Mon Jun 29 18:04:13 2020
+++ pkgsrc/mk/subst.mk	Tue Oct  6 17:48:02 2020
@@ -1,4 +1,4 @@
-# $NetBSD: subst.mk,v 1.102 2020/06/29 18:04:13 rillig Exp $
+# $NetBSD: subst.mk,v 1.103 2020/10/06 17:48:02 rillig Exp $
 #
 # The subst framework replaces text in one or more files in the WRKSRC
 # directory. Packages can define several ``classes'' of replacements.
@@ -21,20 +21,6 @@
 #	Whether to log each changed file as a unified diff, for all
 #	SUBST classes. Defaults to "no".
 #
-# SUBST_NOOP_OK
-#	Whether it is ok to have patterns in SUBST_FILES that don't
-#	contain any of the patterns from SUBST_SED or SUBST_VARS and
-#	thus are not modified at all.
-#
-#	This setting only detects redundant filename patterns.  It does
-#	not detect redundant patterns in SUBST_SED.
-#
-#	Identity substitutions like s|man|man| do not count as no-ops
-#	since their replacement part usually comes from a variable, such
-#	as PKGMANDIR.
-#
-#	Defaults to no.  Will be removed after 2020Q3.
-#
 # Package-settable variables:
 #
 # SUBST_CLASSES
@@ -92,7 +78,15 @@
 #	Whether to allow filename patterns in SUBST_FILES that don't
 #	contain any of the patterns from SUBST_SED.
 #
-#	Defaults to no, since May 2020.
+#	Defaults to no, which means that each filename pattern in
+#	SUBST_FILES must contain any of the patterns from SUBST_SED.
+#
+#	Identity substitutions like s|man|man| do not count as no-ops
+#	since their replacement part usually comes from a variable, such
+#	as PKGMANDIR.
+#
+#	This setting only detects redundant filename patterns.  It does
+#	not detect redundant patterns in SUBST_SED.
 #
 #	Typical reasons to change this to yes are:
 #
@@ -110,10 +104,9 @@
 #
 
 SUBST_SHOW_DIFF?=	no
-SUBST_NOOP_OK?=		no	# will be forced to "no" after 2020Q3
 
 _VARGROUPS+=		subst
-_USER_VARS.subst=	SUBST_SHOW_DIFF SUBST_NOOP_OK
+_USER_VARS.subst=	SUBST_SHOW_DIFF
 _PKG_VARS.subst=	SUBST_CLASSES
 .for c in ${SUBST_CLASSES}
 .  for pv in SUBST_STAGE SUBST_MESSAGE SUBST_FILES SUBST_SED SUBST_VARS	\
@@ -155,7 +148,7 @@ _SUBST_KEEP.${class}?=		LC_ALL=C ${DIFF}
 _SUBST_KEEP.${class}?=		${DO_NADA}
 SUBST_SKIP_TEXT_CHECK.${class}?= \
 				no
-SUBST_NOOP_OK.${class}?=	${SUBST_NOOP_OK}
+SUBST_NOOP_OK.${class}?=	no
 _SUBST_WARN.${class}=		${${SUBST_NOOP_OK.${class}:tl} == yes:?${INFO_MSG}:${WARNING_MSG}} "[subst.mk:${class}]"
 
 .  if !empty(SUBST_SKIP_TEXT_CHECK.${class}:M[Yy][Ee][Ss])


--_----------=_1602006482183640--