Web   ·   Wiki   ·   Activities   ·   Blog   ·   Lists   ·   Chat   ·   Meeting   ·   Bugs   ·   Git   ·   Translate   ·   Archive   ·   People   ·   Donate
summaryrefslogtreecommitdiffstats
path: root/devbot
diff options
context:
space:
mode:
authorDaniel Narvaez <dwnarvaez@gmail.com>2012-11-28 17:14:10 (GMT)
committer Daniel Narvaez <dwnarvaez@gmail.com>2012-11-28 17:14:10 (GMT)
commitd7a1df47e902cca01a1c56cea04cc25722c15948 (patch)
tree1b01e27d32d833028cceb45ab6c586f7782030fe /devbot
parent062b636866f6593d4ecf9eadd849b177b96669c6 (diff)
Return error if build/pull fails
Diffstat (limited to 'devbot')
-rw-r--r--devbot/build.py22
1 files changed, 16 insertions, 6 deletions
diff --git a/devbot/build.py b/devbot/build.py
index 5abe724..7b2e9b4 100644
--- a/devbot/build.py
+++ b/devbot/build.py
@@ -2,7 +2,6 @@ import fnmatch
import os
import multiprocessing
import shutil
-import sys
import subprocess
import time
@@ -16,20 +15,27 @@ def build_one(module_name):
for module in config.load_modules():
if module.name == module_name:
- _build_module(module)
+ return _build_module(module)
+
+ return False
def pull_one(module_name):
environ.setup()
for module in config.load_modules():
if module.name == module_name:
- _pull_module(module)
+ return _pull_module(module)
+
+ return False
def pull():
environ.setup()
for module in config.load_modules():
- _pull_module(module)
+ if not _pull_module(module):
+ return False
+
+ return True
def build():
environ.setup()
@@ -59,7 +65,9 @@ def build():
for module in modules:
log = "build-%s" % time.strftime("%Y%m%d-%H%M%S")
if not _build_module(module, log):
- break
+ return False
+
+ return True
def clean():
_rmtree(config.install_dir)
@@ -99,7 +107,9 @@ def _pull_module(module):
try:
_pull_git_module(module)
except subprocess.CalledProcessError:
- sys.exit(1)
+ return False
+
+ return True
def _build_autotools(module, log):
makefile_path = os.path.join(module.get_build_dir(), "Makefile")