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

Unified Diff: infra/bots/recipe_modules/run/api.py

Issue 2215803002: Move builder_spec, [dm|nanobench]_flags, builder_name_schema to recipes (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Add missing blacklist Created 4 years, 4 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 | « infra/bots/recipe_modules/core/fake_specs.py ('k') | infra/bots/recipe_modules/vars/__init__.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: infra/bots/recipe_modules/run/api.py
diff --git a/infra/bots/recipe_modules/run/api.py b/infra/bots/recipe_modules/run/api.py
index 1784254d6e07b5056d58f3810dda2c19c6c3b5f3..20e82f2b1b12676afdf0b1ee32cdda7c656a6a1f 100644
--- a/infra/bots/recipe_modules/run/api.py
+++ b/infra/bots/recipe_modules/run/api.py
@@ -82,16 +82,6 @@ class SkiaStepApi(recipe_api.RecipeApi):
if fail_build_on_failure:
self._failed.append(e)
- def json_from_file(self, filename, cwd, builder_name, test_data):
- """Execute the given script to obtain JSON data."""
- return self.m.python(
- 'exec %s' % self.m.path.basename(filename),
- filename,
- args=[self.m.json.output(), builder_name],
- step_test_data=lambda: self.m.json.test_api.output(test_data),
- cwd=cwd,
- infra_step=True).json.output
-
def copy_build_products(self, src, dst):
"""Copy whitelisted build products from src to dst."""
self.m.python.inline(
« no previous file with comments | « infra/bots/recipe_modules/core/fake_specs.py ('k') | infra/bots/recipe_modules/vars/__init__.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698