[PATCH 1/2] maint.mk: add leading "@" to quiet new "make syntax-check" rule

[Jim Meyering]

I tried out the latest from gnulib and saw that "make syntax-check"
was accidentally made too verbose.

Since this has happened too frequently, I've written
a syntax-check rule (we call it sc_maint) for gnulib itself
to help prevent a repeat.

>From 1f2e7fd8897db561009be90c66a70354aaa6a4b8 Mon Sep 17 00:00:00 2001
From: Jim Meyering <address@hidden>
Date: Sun, 15 Jul 2012 14:40:29 +0200
Subject: [PATCH 1/2] maint.mk: add leading "@" to quiet new "make
 syntax-check" rule

* top/maint.mk (sc_prohibit_defined_have_decl_tests): Add "@".
---
 ChangeLog    | 5 +++++
 top/maint.mk | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/ChangeLog b/ChangeLog
index 43572fa..4260f45 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2012-07-15  Jim Meyering  <address@hidden>
+
+       maint.mk: add leading "@" to quiet new "make syntax-check" rule
+       * top/maint.mk (sc_prohibit_defined_have_decl_tests): Add "@".
+
 2012-07-13  Eric Blake  <address@hidden>

        maint.mk: new syntax check for HAVE_DECL checks
diff --git a/top/maint.mk b/top/maint.mk
index 5ca1a31..e527c61 100644
--- a/top/maint.mk
+++ b/top/maint.mk
@@ -778,7 +778,7 @@ sc_prohibit_always_true_header_tests:
          $(_sc_search_regexp)

 sc_prohibit_defined_have_decl_tests:
-       prohibit='#[     ]*if(n?def|.*\<defined)\>[      (]+HAVE_DECL_' \
+       @prohibit='#[    ]*if(n?def|.*\<defined)\>[      (]+HAVE_DECL_' \
        halt='$(ME): HAVE_DECL macros are always defined'               \
          $(_sc_search_regexp)

--
1.7.11.2.194.g7bdb748


>From 696aa74b38f8c109c21e4c01194faab53fb4cefc Mon Sep 17 00:00:00 2001
From: Jim Meyering <address@hidden>
Date: Sun, 15 Jul 2012 15:15:46 +0200
Subject: [PATCH 2/2] maint: require that each sc_... command start with "@"

* Makefile (sc_prohibit_sc_omitted_at): New rule so that
"make sc_maint" helps us avoid this nit.
---
 ChangeLog |  6 ++++++
 Makefile  | 30 +++++++++++++++++++++---------
 2 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 4260f45..a0fa30a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2012-07-15  Jim Meyering  <address@hidden>

+       maint: require that each sc_... command start with "@"
+       * Makefile (sc_prohibit_sc_omitted_at): New rule so that
+       "make sc_maint" helps us avoid this nit.
+
+2012-07-15  Jim Meyering  <address@hidden>
+
        maint.mk: add leading "@" to quiet new "make syntax-check" rule
        * top/maint.mk (sc_prohibit_defined_have_decl_tests): Add "@".

diff --git a/Makefile b/Makefile
index 675c8e0..10af1b9 100644
--- a/Makefile
+++ b/Makefile
@@ -26,14 +26,14 @@ syntax-check-rules := $(sort $(shell sed -n 
's/^\(sc_[a-zA-Z0-9_-]*\):.*/\1/p'\
 check: $(syntax-check-rules)

 sc_prefer_ac_check_funcs_once:
-       if test -d .git; then                                           \
+       @if test -d .git; then                                          \
          git grep -w -l AC_CHECK_FUNCS modules                         \
            && { echo use AC_CHECK_FUNCS_ONCE, not AC_CHECK_FUNCS       \
                   in modules/ 1>&2; exit 1; } || :                     \
        else :; fi

 sc_prohibit_leading_TABs:
-       if test -d .git; then                                           \
+       @if test -d .git; then                                          \
          git grep -l '^ *      ' lib m4 tests                          \
            | grep -Ev '^lib/reg|Makefile|test-update-copyright'        \
            | grep .                                                    \
@@ -42,16 +42,28 @@ sc_prohibit_leading_TABs:
        else :; fi

 sc_prohibit_augmenting_PATH_via_TESTS_ENVIRONMENT:
-       if test -d .git; then                                           \
+       @if test -d .git; then                                          \
          url=http://thread.gmane.org/gmane.comp.lib.gnulib.bugs/22874; \
          git grep '^[   ]*TESTS_ENVIRONMENT += PATH=' modules          \
            && { printf '%s\n' 'Do not augment PATH via TESTS_ENVIRONMENT;' \
                 "  see <$$url>" 1>&2; exit 1; } || :                   \
        else :; fi

+# It's easy to forget the noise-suppressing "@" at the beginning
+# of each sc_ rule.  Check for it both in maint.mk and in this file.
+sc_prohibit_sc_omitted_at:
+       @if test -d .git; then                                          \
+         git grep -n -A1 '^sc_[[:alnum:]_-]*:' top/maint.mk Makefile   \
+            | grep -vE ':sc_|[0-9][-]  @|--$$'                         \
+            | sed 's/-\([0-9][0-9]*\)-/:\1:/'                          \
+           | grep .                                                    \
+           && { printf '*** %s\n' 'oops; missing "@"'                  \
+                1>&2; exit 1; } || :                                   \
+       else :; fi
+
 # Run all maint.mk syntax-check tests on gnulib's sources.
 sc_maint:
-       rm -f maint.mk; ln -s top/maint.mk maint.mk
+       @rm -f maint.mk; ln -s top/maint.mk maint.mk
        $(MAKE) -s srcdir=. gnulib_dir=. _build-aux=build-aux \
             -f cfg.mk -f maint.mk syntax-check
        rm -f maint.mk
@@ -83,7 +95,7 @@ allow_AC_LIBOBJ =     \
 allow_AC_LIBOBJ_or := $(shell echo $(allow_AC_LIBOBJ) | tr -s ' ' '|')

 sc_prohibit_AC_LIBOBJ_in_m4:
-       url=http://article.gmane.org/gmane.comp.lib.gnulib.bugs/26995;  \
+       @url=http://article.gmane.org/gmane.comp.lib.gnulib.bugs/26995; \
        if test -d .git; then                                           \
          git ls-files m4                                               \
             | grep -Ev '^m4/($(allow_AC_LIBOBJ_or))\.m4$$'             \
@@ -93,7 +105,7 @@ sc_prohibit_AC_LIBOBJ_in_m4:
        else :; fi

 sc_pragma_columns:
-       if test -d .git; then                                           \
+       @if test -d .git; then                                          \
          git ls-files|grep '\.in\.h$$'                                 \
              | xargs grep -l 'address@hidden@'                 \
              | xargs grep -L 'address@hidden@'                 \
@@ -107,7 +119,7 @@ sc_pragma_columns:
 # Verify that certain (for now, only Jim Meyering and Eric Blake's)
 # *.c files are consistently cpp indented.
 sc_cpp_indent_check:
-       ./gnulib-tool --extract-filelist \
+       @./gnulib-tool --extract-filelist \
            $$(cd ./modules; grep -ilrE '(meyering|blake)' .) \
          | sort -u \
          | grep '\.c$$' \
@@ -119,7 +131,7 @@ sc_cpp_indent_check:
 # Extract the symbols from the .h file and compare with the list of
 # symbols extracted from the rule in maint.mk.
 sc_check_sym_list:
-       i=lib/intprops.h; \
+       @i=lib/intprops.h; \
        diff -u <(perl -lne '/^# *define ([A-Z]\w+)\(/ and print $$1' $$i|fmt) \
          <(sed -n /^_intprops_name/,/^_intprops_syms_re/p top/maint.mk \
            |sed '/^_/d;s/^  //;s/      *\\$$//')
@@ -127,7 +139,7 @@ sc_check_sym_list:
 # Ensure that the copyright statements in files and in the module descriptions
 # are consistent.
 sc_check_copyright:
-       ./check-copyright
+       @./check-copyright

 # Regenerate some files that are stored in the repository.
 regen: MODULES.html
--
1.7.11.2.194.g7bdb748