buildman: Don't prune output space for 'current source' build
authorSimon Glass <sjg@chromium.org>
Tue, 2 Dec 2014 00:33:53 +0000 (17:33 -0700)
committerSimon Glass <sjg@chromium.org>
Thu, 15 Jan 2015 05:16:52 +0000 (21:16 -0800)
This is not needed since we always do a full (non-incremental) build. Also
it might be dangerous since it will try to delete everything below the
base directory.

Fix this potentially nasty bug.

Signed-off-by: Simon Glass <sjg@chromium.org>
tools/buildman/builder.py

index 7002034221d12789473d33e7f62cb21cbe296be7..05ebfd20219744e523930883510a5420c627c4e8 100644 (file)
@@ -1115,6 +1115,8 @@ class Builder:
         create. Having left over directories is confusing when the user wants
         to check the output manually.
         """
+        if not self.commits:
+            return
         dir_list = []
         for commit_upto in range(self.commit_count):
             dir_list.append(self._GetOutputDir(commit_upto))