diff options
author | Daniel Narvaez <dwnarvaez@gmail.com> | 2012-12-25 01:22:46 (GMT) |
---|---|---|
committer | Daniel Narvaez <dwnarvaez@gmail.com> | 2012-12-25 01:22:46 (GMT) |
commit | 4f9a11409b3b3ea9d2415472b29a481834089a1e (patch) | |
tree | abdb572e004d15ac6f82ec121fae2d7acda84c91 /devbot/build.py | |
parent | 5a97aac1de8b120747fc79b20dfbe4db81116515 (diff) |
Store built module commit in a dict
So that we can store more info
Diffstat (limited to 'devbot/build.py')
-rw-r--r-- | devbot/build.py | 11 |
1 files changed, 3 insertions, 8 deletions
diff --git a/devbot/build.py b/devbot/build.py index 9823310..6a7f387 100644 --- a/devbot/build.py +++ b/devbot/build.py @@ -52,12 +52,7 @@ def build(): skipped = [] for module in modules[:]: - new_commit_id = module.get_commit_id() - if new_commit_id is None: - break - - old_commit_id = state.get_built_commit_id(module) - if old_commit_id == new_commit_id: + if state.check_built_module(module): modules.pop(0) skipped.append(module.name) else: @@ -68,7 +63,7 @@ def build(): print "\n".join(skipped) for module in modules: - state.remove_built_commit_id(module) + state.remove_built_module(module) for module in modules: if not _build_module(module, config.get_log_path("build")): @@ -226,7 +221,7 @@ def _build_module(module, log=None): except subprocess.CalledProcessError: return False - state.touch_built_commit_id(module) + state.touch_built_module(module) return True |