maint.mk: improve the public-submodule-commit rule

* top/maint.mk (public-submodule-commit): Prefix with $(AM_V_GEN),
to suppress printing of its commands... unless V=1.
Add git submodule's --quiet option to suppress printing of e.g.,
"Entering gnulib" output.
"cd" into $(srcdir) before running git submodule.

diff --git a/ChangeLog b/ChangeLog
index 32dc93c1dd1f6eb91229c8ea88b466c7b44aa276..9c125c9eb2767bcd49ba6eda61b5f145518e051c 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,12 @@
+2011-01-20  Jim Meyering  <meyering@redhat.com>
+
+       maint.mk: improve the public-submodule-commit rule
+       * top/maint.mk (public-submodule-commit): Prefix with $(AM_V_GEN),
+       to suppress printing of its commands... unless V=1.
+       Add git submodule's --quiet option to suppress printing of e.g.,
+       "Entering gnulib" output.
+       "cd" into $(srcdir) before running git submodule.
+
 2011-01-20  Bruno Haible  <bruno@clisp.org>
 
        include_next: Fix bug introduced on 2011-01-18.

diff --git a/top/maint.mk b/top/maint.mk
index ae280f648581df4232ac7467828386c5eb1bd5c2..25f4736a3b42eb74cc1919ed32c74e53d7a4b526 100644 (file)
--- a/top/maint.mk
+++ b/top/maint.mk
@@ -1125,8 +1125,9 @@ submodule-checks ?= no-submodule-changes public-submodule-commit
 # cannot be built from a fresh clone.
 .PHONY: public-submodule-commit
 public-submodule-commit:
-       if test -d $(srcdir)/.git; then                                 \
-         git submodule foreach 'test $$(git rev-parse origin)'         \
+       $(AM_V_GEN)if test -d $(srcdir)/.git; then                      \
+         cd $(srcdir) &&                                               \
+         git submodule --quiet foreach 'test $$(git rev-parse origin)' \
              = '"$$(git merge-base --independent origin $$sha1)"'      \
            || { echo '$(ME): found non-public submodule commit' >&2;   \
                 exit 1; };                                             \