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

Unified Diff: scripts/slave/recipe_modules/gsutil/api.py

Issue 2035863004: gsutil: use depot_tools.gsutil_py_path instead of path['depot_tools'] (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 4 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
Index: scripts/slave/recipe_modules/gsutil/api.py
diff --git a/scripts/slave/recipe_modules/gsutil/api.py b/scripts/slave/recipe_modules/gsutil/api.py
index e65ba6ec588d1949eed4dfeebe5a28906af48c15..3aafe2e2de39843be975c89b5a12c9f3f0080181 100644
--- a/scripts/slave/recipe_modules/gsutil/api.py
+++ b/scripts/slave/recipe_modules/gsutil/api.py
@@ -28,7 +28,7 @@ class GSUtilApi(recipe_api.RecipeApi):
name = (t for t in cmd if not t.startswith('-')).next()
full_name = 'gsutil ' + name
- gsutil_path = self.m.path['depot_tools'].join('gsutil.py')
+ gsutil_path = self.m.depot_tools.gsutil_py_path
tandrii(chromium) 2016/06/03 13:28:17 why not move this whole recipe into depot_tools? S
tandrii(chromium) 2016/06/03 13:48:58 discussed in IM: we postpone the actual refactorin
cmd_prefix = []
if use_retry_wrapper:
« no previous file with comments | « scripts/slave/recipe_modules/gsutil/__init__.py ('k') | scripts/slave/recipe_modules/gsutil/example.expected/basic.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698