Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1414)

Unified Diff: scripts/slave/recipes/bisect.py

Issue 1185693002: Move builders.py and steps.py to chromium_tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Address review comments. Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « scripts/slave/recipes/android/perf.py ('k') | scripts/slave/recipes/bisection/desktop_bisect.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipes/bisect.py
diff --git a/scripts/slave/recipes/bisect.py b/scripts/slave/recipes/bisect.py
index fc595ad61ee2d32ab5915d93721894322c85a9bb..9ea252bbd8d199416cd2e8ab07ef86a7b311a796 100644
--- a/scripts/slave/recipes/bisect.py
+++ b/scripts/slave/recipes/bisect.py
@@ -234,6 +234,8 @@ def _get_step_data_for_revision(api, revision_data, broken_cp=None,
def _ensure_checkout(api):
mastername = api.properties.get('mastername')
buildername = api.properties.get('buildername')
+ # TODO(akuegel): Explicitly load the configs for the builders and don't rely
+ # on builders.py in chromium_tests recipe module.
api.chromium_tests.configure_build(mastername, buildername)
api.chromium_tests.prepare_checkout(mastername, buildername)
« no previous file with comments | « scripts/slave/recipes/android/perf.py ('k') | scripts/slave/recipes/bisection/desktop_bisect.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698