Change release procedure: tag *before* running e.g., "make stable".
authorJim Meyering <meyering@redhat.com>
Sat, 1 Dec 2007 18:52:45 +0000 (19:52 +0100)
committerJim Meyering <meyering@redhat.com>
Sat, 1 Dec 2007 19:04:44 +0000 (20:04 +0100)
* Makefile.maint (vc-dist): Don't tag.  Now, you must apply the tag
before running "make beta", "make stable", etc.
(vc-tag-check): Remove rule.  No longer makes sense.

ChangeLog
Makefile.maint

index 4c5e129a08a3ac11d116f8755753d279b69c57ac..3d0f9af8a76b47aacfcef419917e72f3629343c6 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,10 @@
 2007-12-01  Jim Meyering  <meyering@redhat.com>
 
+       Change release procedure: tag *before* running e.g., "make stable".
+       * Makefile.maint (vc-dist): Don't tag.  Now, you must apply the tag
+       before running "make beta", "make stable", etc.
+       (vc-tag-check): Remove rule.  No longer makes sense.
+
        * Makefile.maint (vc-tag-check): Correct check for existing git tag.
 
        Defer NEWS and ChangeLog checks until after the version-changing tag.
index 7a7326d4b88068c4a128ee893d29cc389c04bafe..e9ab2e6b0d99c2db1cbfb0c4f65f8b05dc9a0284 100644 (file)
@@ -513,22 +513,6 @@ copyright-check:
               exit 1; }; \
        fi
 
-
-# Sanity checks with the repository.
-# Abort early if this tag has already been used.
-vc-tag-check:
-       used=no;                                                        \
-       if $(VC) --help | grep CVS; then                                \
-         $(CVS) -n log -h README|grep -e $(this-vc-tag-regexp): >/dev/null \
-           && used=yes;                                                \
-       else                                                            \
-         test -n `$(GIT) tag -l '$(this-vc-tag)'` && used=yes;         \
-       fi;                                                             \
-       if test "$$used" = yes; then                                    \
-         echo "$(this-vc-tag) has already been used; not tagging" 1>&2; \
-         exit 1;                                                       \
-       fi
-
 vc-diff-check:
        $(VC) diff > vc-diffs || :
        if test -s vc-diffs; then                               \
@@ -539,18 +523,16 @@ vc-diff-check:
          rm vc-diffs;                                          \
        fi
 
-cvs-check: vc-diff-check vc-tag-check
+cvs-check: vc-diff-check
 
 maintainer-distcheck:
        $(MAKE) distcheck
        $(MAKE) my-distcheck
 
 
-# Tag before making distribution.  Also, don't make a distribution if
-# checks fail.  Also, make sure the NEWS file is up-to-date.
-# FIXME: use dist-hook/my-dist like distcheck-hook/my-distcheck.
+# Don't make a distribution if checks fail.
+# Also, make sure the NEWS file is up-to-date.
 vc-dist: $(local-check) cvs-check maintainer-distcheck
-       $(VC-tag) $(this-vc-tag)
        $(MAKE) dist
 
 # Use this to make sure we don't run these programs when building