buildman: Drop the 'active' flag in the builder
This serves no real purpose, since when we are not active, we exit. Drop it.
Signed-off-by: Simon Glass <sjg@chromium.org>
diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 44d1cfa..5addbca 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -127,7 +127,6 @@
"""Class for building U-Boot for a particular commit.
Public members: (many should ->private)
- active: True if the builder is active and has not been stopped
already_done: Number of builds already completed
base_dir: Base directory to use for builder
checkout: True to check out source, False to skip that step.
@@ -235,7 +234,6 @@
self.base_dir = base_dir
self._working_dir = os.path.join(base_dir, '.bm-work')
self.threads = []
- self.active = True
self.do_make = self.Make
self.gnu_make = gnu_make
self.checkout = checkout
@@ -390,11 +388,6 @@
if result:
target = result.brd.target
- if result.return_code < 0:
- self.active = False
- command.StopAll()
- return
-
self.upto += 1
if result.return_code != 0:
self.fail += 1
diff --git a/tools/buildman/builderthread.py b/tools/buildman/builderthread.py
index c512d3b..af4d15a 100644
--- a/tools/buildman/builderthread.py
+++ b/tools/buildman/builderthread.py
@@ -473,14 +473,6 @@
alive = True
while True:
job = self.builder.queue.get()
- if self.builder.active and alive:
+ if alive:
self.RunJob(job)
- '''
- try:
- if self.builder.active and alive:
- self.RunJob(job)
- except Exception as err:
- alive = False
- print err
- '''
self.builder.queue.task_done()