diff options
author | borenet <borenet@chromium.org> | 2016-08-04 11:33:01 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-08-04 11:33:02 -0700 |
commit | c7233ef925157d0c881a1caebe7836e3823c88d5 (patch) | |
tree | 44ebec644e7283c3801f08ca8d2137b8716be6bb /infra/bots/recipe_modules/core | |
parent | 538d5b68e8b6a8745ac8947f15abf1c7c6de8f18 (diff) |
[recipes] Stop pretending that buildbot_spec is a step, simplify
Since we no longer have to call out to obtain the buildbot_spec, merge
the two functions in vars API. As a side effect, this applies
default_env to the sync steps as well, which shouldn't have an
appreciable effect on bot behavior.
BUG=skia:5578
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2209343002
Review-Url: https://codereview.chromium.org/2209343002
Diffstat (limited to 'infra/bots/recipe_modules/core')
-rw-r--r-- | infra/bots/recipe_modules/core/api.py | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/infra/bots/recipe_modules/core/api.py b/infra/bots/recipe_modules/core/api.py index 377026432e..e65073cb36 100644 --- a/infra/bots/recipe_modules/core/api.py +++ b/infra/bots/recipe_modules/core/api.py @@ -25,13 +25,6 @@ class SkiaApi(recipe_api.RecipeApi): # Check out the Skia code. self.checkout_steps() - # Obtain the spec for this builder from the Skia repo. Use it to set more - # properties. - builder_spec = self.m.vars.get_builder_spec(self.m.vars.builder_name) - - # Continue setting up vars with the builder_spec. - self.m.vars.update_with_builder_spec(builder_spec) - if not self.m.path.exists(self.m.vars.tmp_dir): self.m.run.run_once(self.m.file.makedirs, 'tmp_dir', |