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

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

Issue 2221413002: Move ct_skps recipe from tools repo to Skia repo (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: Changes 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
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 20e82f2b1b12676afdf0b1ee32cdda7c656a6a1f..eb0f8e679a1231a66b79abb1953d3253c2eae11b 100644
--- a/infra/bots/recipe_modules/run/api.py
+++ b/infra/bots/recipe_modules/run/api.py
@@ -12,6 +12,8 @@ from recipe_engine import recipe_api
BUILD_PRODUCTS_ISOLATE_WHITELIST = [
'dm',
'dm.exe',
+ 'get_images_from_skps',
+ 'get_images_from_skps.exe',
'nanobench',
'nanobench.exe',
'*.so',
@@ -62,8 +64,8 @@ class SkiaStepApi(recipe_api.RecipeApi):
def rmtree(self, path):
"""Wrapper around api.file.rmtree with environment fix."""
env = {}
- env['PYTHONPATH'] = str(self.m.path['checkout'].join(
- 'infra', 'bots', '.recipe_deps', 'build', 'scripts'))
+ env['PYTHONPATH'] = str(self.m.path['slave_build'].join(
+ 'skia', 'infra', 'bots', '.recipe_deps', 'build', 'scripts'))
borenet 2016/08/09 17:10:21 Can you move this change into a separate CL and la
rmistry 2016/08/09 18:19:11 Done: https://codereview.chromium.org/2221423002/#
self.m.file.rmtree(self.m.path.basename(path),
path,
env=env,

Powered by Google App Engine
This is Rietveld 408576698