maint.mk: don't silently disable project-specific syntax-check rules

* top/maint.mk (_prohibit_regexp): Define, to help people realize
that they need to convert their project-specific syntax-check rules
to use the new _sc_search_regexp.

diff --git a/ChangeLog b/ChangeLog
index 3f3330256dea7f2076541761d85c0f86e072c039..36443a30715e71696adb87897730b956ef39b834 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-04-03  Jim Meyering  <meyering@redhat.com>
+
+       maint.mk: don't silently disable project-specific syntax-check rules
+       * top/maint.mk (_prohibit_regexp): Define, to help people realize
+       that they need to convert their project-specific syntax-check rules
+       to use the new _sc_search_regexp.
+
 2010-04-03  Bruno Haible  <bruno@clisp.org>
 
        fchdir: Fix regression introduced on 2010-03-08.

diff --git a/top/maint.mk b/top/maint.mk
index 13a0438f90d9a175bb4a14221d1659ed8aa22d09..3038680e3e0783cce3fdc149800314e38ac8f487 100644 (file)
--- a/top/maint.mk
+++ b/top/maint.mk
@@ -189,6 +189,15 @@ define _sc_say_and_exit
    { echo -e "$(ME): $$msg" 1>&2; exit 1; };
 endef
 
+# _sc_search_regexp used to be named _prohibit_regexp.  However,
+# upgrading to the new definition and leaving the old name undefined
+# would usually convert each custom rule using $(_prohibit_regexp)
+# (usually defined in cfg.mk) into a no-op.  This definition ensures
+# that people know right away if they're still using the old name.
+# FIXME: remove in 2012.
+_prohibit_regexp = \
+  $(error '*** you need to s/_prohibit_regexp/_sc_search_regexp/, and adapt')
+
 define _sc_search_regexp
    dummy=; : so we do not need a semicolon before each use;            \
                                                                        \