Merge branch 'buildbot-0.8.0'

* buildbot-0.8.0:
  Git: add missing "submodules" documentation.
  Git: document 'shallow' option.
  Git: add test for shallow clones.
  Git: be shallow on an initial clone.

buildbot/broken_test/vc/test_vc.py

@@ -3136,6 +3136,12 @@ def testCheckout(self):
         d = self.do_vctest()
         return d
 
+    def testCheckoutShallow(self):
+        self.helper.vcargs = { 'repourl': self.helper.gitrepo,
+                               'shallow': True }
+        d = self.do_vctest()
+        return d
+
     def testPatch(self):
         self.helper.vcargs = { 'repourl': self.helper.gitrepo,
                                'branch': "master" }

buildbot/slave/commands/vcs.py

@@ -1026,8 +1026,21 @@ def _didInit(self, res):
         return self.doVCUpdate()
 
     def doVCFull(self):
-        os.makedirs(self._fullSrcdir())
-        return self._dovccmd(['init'], self._didInit)
+        # If they didn't ask for a specific revision, we can get away with a
+        # shallow clone.
+        if not self.args.get('revision') and self.args.get('shallow'):
+            cmd = [self.vcexe, 'clone', '--depth', '1', self.repourl,
+                   self._fullSrcdir()]
+            c = ShellCommand(self.builder, cmd, self.builder.basedir,
+                             sendRC=False, timeout=self.timeout,
+                             maxTime=self.maxTime, usePTY=False)
+            self.command = c
+            cmdexec = c.start()
+            cmdexec.addCallback(self._didInit)
+            return cmdexec
+        else:
+            os.makedirs(self._fullSrcdir())
+            return self._dovccmd(['init'], self._didInit)
 
     def parseGotRevision(self):
         command = ['rev-parse', 'HEAD']

buildbot/steps/source.py

@@ -705,6 +705,7 @@ def __init__(self, repourl,
                  branch="master",
                  submodules=False,
                  ignore_ignores=None,
+                 shallow=False,
                  **kwargs):
         """
         @type  repourl: string
@@ -719,17 +720,21 @@ def __init__(self, repourl,
         @param submodules: Whether or not to update (and initialize)
                        git submodules.
 
+        @type  shallow: boolean
+        @param shallow: Use a shallow or clone, if possible
         """
         Source.__init__(self, **kwargs)
         self.addFactoryArguments(repourl=repourl,
                                  branch=branch,
                                  submodules=submodules,
                                  ignore_ignores=ignore_ignores,
+                                 shallow=shallow,
                                  )
         self.args.update({'repourl': repourl,
                           'branch': branch,
                           'submodules': submodules,
                           'ignore_ignores': ignore_ignores,
+                          'shallow': shallow,
                           })
 
     def computeSourceRevision(self, changes):

docs/cfg-buildsteps.texinfo

@@ -860,6 +860,14 @@ branch will be used.
 (optional): when purging changes, don't use .gitignore and
 .git/info/exclude.
 
+@item submodules
+(optional): when initializing/updating a Git repository, this decides whether
+or not buildbot should consider git submodules.  Default: False.
+
+@item shallow
+(optional): instructs git to attempt shallow clones (@code{--depth 1}).  If the
+user/scheduler asks for a specific revision, this parameter is ignored.
+
 @end table