vr.common 5.4.1 adds support for accepting None as default for rev pa…

…rameter to Repo.update

setup.py

@@ -37,7 +37,7 @@
         'path.py>=7.1',
         'yg.lockfile',
         'jaraco.itertools',
-        'vr.common>=4.3.0',
+        'vr.common>=5.4.1',
     ],
     extras_require={
         'testing': [

vr/builder/models.py

@@ -10,7 +10,7 @@
 import yg.lockfile
 
 from vr.common import repo
-from vr.common.utils import run, mkdir, chdir
+from vr.common.utils import run, mkdir
 from vr.common.paths import VR_ROOT
 
 from vr.builder.slugignore import clean_slug_dir
@@ -75,29 +75,6 @@ def release(self, app):
                                          "buildpack" % (app, self.basename))
         return yaml.safe_load(result.output)
 
-    def update(self, rev=None):
-        """
-        Override Repo.update to provide default rev when none is provided.
-        """
-        if not os.path.exists(self.folder):
-            self.clone()
-
-        # account for self.fragment=='' case
-        rev = rev or self.fragment or None
-
-        if self.vcs_type == 'git':
-            if rev is None:
-                # Git needs special handling if no rev is passed.
-                # Don't ask for a revision, just do a pull.
-                with chdir(self.folder):
-                    self.run('git fetch --tags')
-                    self.run('git pull origin master')
-            else:
-                return super(BuildPack, self).update(rev)
-        elif self.vcs_type == 'hg':
-            # Are there actually any mercurial buildpacks?
-            return super(BuildPack, self).update(rev or 'tip')
-
 
 class App(repo.Repo):
     """