fullrelease: share vcs between stages.

Otherwise we would ask 'do you want to change to the root of the repo' three times.

zest/releaser/baserelease.py

@@ -11,8 +11,13 @@
 
 class Basereleaser(object):
 
-    def __init__(self):
-        self.vcs = choose.version_control()
+    def __init__(self, vcs=None):
+        if vcs is None:
+            self.vcs = choose.version_control()
+        else:
+            # In a fullrelease, we share the determined vcs between
+            # prerelease, release and postrelease.
+            self.vcs = vcs
         self.data = {'workingdir': self.vcs.workingdir,
                      'reporoot': self.vcs.reporoot,
                      'name': self.vcs.name}

zest/releaser/fullrelease.py

@@ -18,13 +18,18 @@ def main():
     utils.configure_logging()
     logger.info('Starting prerelease.')
     original_dir = os.getcwd()
-    prerelease.main()
-    os.chdir(original_dir)
+    # prerelease
+    prereleaser = prerelease.Prereleaser()
+    prereleaser.run()
     logger.info('Starting release.')
-    tagdir = release.main(return_tagdir=True)
-    os.chdir(original_dir)
+    # release
+    releaser = release.Releaser(vcs=prereleaser.vcs)
+    releaser.run()
+    tagdir = releaser.data.get('tagdir')
     logger.info('Starting postrelease.')
-    postrelease.main()
+    # postrelease
+    postreleaser = postrelease.Postreleaser(vcs=releaser.vcs)
+    postreleaser.run()
     os.chdir(original_dir)
     logger.info('Finished full release.')
     if tagdir:

zest/releaser/postrelease.py

@@ -43,8 +43,8 @@ class Postreleaser(baserelease.Basereleaser):
 
     """
 
-    def __init__(self):
-        baserelease.Basereleaser.__init__(self)
+    def __init__(self, vcs=None):
+        baserelease.Basereleaser.__init__(self, vcs=vcs)
         # Prepare some defaults for potential overriding.
         self.data.update(dict(
             nothing_changed_yet=NOTHING_CHANGED_YET,

zest/releaser/prerelease.py

@@ -47,8 +47,8 @@ class Prereleaser(baserelease.Basereleaser):
 
     """
 
-    def __init__(self):
-        baserelease.Basereleaser.__init__(self)
+    def __init__(self, vcs=None):
+        baserelease.Basereleaser.__init__(self, vcs=vcs)
         # Prepare some defaults for potential overriding.
         self.data.update(dict(
             today=datetime.datetime.today().strftime('%Y-%m-%d'),

zest/releaser/release.py

@@ -47,8 +47,8 @@ def package_in_pypi(package):
 class Releaser(baserelease.Basereleaser):
     """Release the project by tagging it and optionally uploading to pypi."""
 
-    def __init__(self):
-        baserelease.Basereleaser.__init__(self)
+    def __init__(self, vcs=None):
+        baserelease.Basereleaser.__init__(self, vcs=vcs)
         # Prepare some defaults for potential overriding.
         self.data.update(dict(
             # Nothing yet
@@ -310,15 +310,11 @@ def datacheck(data):
     utils.is_data_documented(data, documentation=DATA)
 
 
-def main(return_tagdir=False):
+def main():
     utils.parse_options()
     utils.configure_logging()
     releaser = Releaser()
     releaser.run()
-    if return_tagdir:
-        # At the end, for the benefit of fullrelease.
-        return releaser.data.get('tagdir')
-    else:
-        tagdir = releaser.data.get('tagdir')
-        if tagdir:
-            logger.info("Reminder: tag checkout is in %s", tagdir)
+    tagdir = releaser.data.get('tagdir')
+    if tagdir:
+        logger.info("Reminder: tag checkout is in %s", tagdir)