# HG changeset patch
# User Jim Meyering <meyering@redhat.com>
# Date 1241295610 -7200
# Node ID 9f608fba8b1c212ef584ad012822f4733b9ae6c3
# Parent  bcb5cdd9a18c549c177706c04a4ccd2e8d983344
build: ensure that a release build fails when a submodule is unclean

* top/maint.mk (no-submodule-changes): New rule.
(alpha beta major): Depend on it.

diff --git a/ChangeLog b/ChangeLog
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2009-05-02  Jim Meyering  <meyering@redhat.com>
+
+	build: ensure that a release build fails when a submodule is unclean
+	* top/maint.mk (no-submodule-changes): New rule.
+	(alpha beta major): Depend on it.
+
 2009-05-02  Bruno Haible  <bruno@clisp.org>
 
 	Remove incompatibility between modules fnmatch-posix and fnmatch-gnu.
diff --git a/top/maint.mk b/top/maint.mk
--- a/top/maint.mk
+++ b/top/maint.mk
@@ -676,9 +676,21 @@
     '* cfg.mk (old_NEWS_hash): Auto-update.'
 endef
 
+.PHONY: no-submodule-changes
+no-submodule-changes:
+	if test -d .git; then						\
+	  diff=$$(git submodule -q foreach git diff-index --name-only HEAD) \
+	    || exit 1;							\
+	  case $$diff in '') ;;						\
+	    *) echo '$(ME): submodule files are locally modified:';	\
+		echo "$$diff"; exit 1;; esac;				\
+	else								\
+	  : ;								\
+	fi
+
 .PHONY: alpha beta major
 ALL_RECURSIVE_TARGETS += alpha beta major
-alpha beta major: $(local-check) writable-files
+alpha beta major: $(local-check) writable-files no-submodule-changes
 	test $@ = major						\
 	  && { echo $(VERSION) | grep -E '^[0-9]+(\.[0-9]+)+$$'	\
 	       || { echo "invalid version string: $(VERSION)" 1>&2; exit 1;};}\