Fix release.py to allow uncommented initial _VERSION.
authorW. Trevor King <wking@drexel.edu>
Thu, 16 Feb 2012 16:57:31 +0000 (11:57 -0500)
committerW. Trevor King <wking@drexel.edu>
Thu, 16 Feb 2012 16:57:35 +0000 (11:57 -0500)
I was in favor of always leaving _VERSION commented in the trunk,
since released branches should fork off the trunk:

  trunkA -> trunkB -> trunkC -> trunkD
   `-> 1.0.0           `-> 1.1.0
        `-> 1.0.1           `-> 1.1.1

But that doesn't seem to have been how things have worked out in BE.
In any case, you will need to release on top of a previous release
(e.g. 1.0.1 on top of 1.0.0 in my above example), so we cannot depend
on an initial comment character before _VERSION.

release.py

index 0e144d8b55063c5e92c6cad76ce8a65e88e5225c..74648714428d932698f9e0766645866f11cd61cf 100755 (executable)
@@ -66,7 +66,7 @@ def pending_changes():
 
 def set_release_version(tag):
     print "set libbe.version._VERSION = '%s'" % tag
-    invoke(['sed', '-i', "s/^*_VERSION *=.*/_VERSION = '%s'/" % tag,
+    invoke(['sed', '-i', "s/^[# ]*_VERSION *=.*/_VERSION = '%s'/" % tag,
             os.path.join('libbe', 'version.py')])
 
 def remove_makefile_libbe_version_dependencies(filename):