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

Unified Diff: recipe_modules/gclient/example.expected/basic.json

Issue 1676473002: gclient: add depot_tools to PATH (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: Created 4 years, 10 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: recipe_modules/gclient/example.expected/basic.json
diff --git a/recipe_modules/gclient/example.expected/basic.json b/recipe_modules/gclient/example.expected/basic.json
index 15d9d2f40d7fd6fdda79e56cce51b4cbe48c1ff6..46c74809df03069d97da2ffd51b9e6ee735263e1 100644
--- a/recipe_modules/gclient/example.expected/basic.json
+++ b/recipe_modules/gclient/example.expected/basic.json
@@ -9,6 +9,9 @@
"cache_dir = '[ROOT]/git_cache'\nsolutions = [{'deps_file': '.DEPS.git', 'managed': True, 'name': 'src', 'url': 'https://chromium.googlesource.com/chromium/src.git'}]"
],
"cwd": "[SLAVE_BUILD]",
+ "env": {
+ "PATH": "%(PATH)s:RECIPE_PACKAGE[depot_tools]"
Michael Achenbach 2016/02/05 11:56:41 This %(PATH)s doesn't really look right.
+ },
"name": "gclient setup"
},
{
@@ -30,6 +33,9 @@
"/path/to/tmp/json"
],
"cwd": "[SLAVE_BUILD]",
+ "env": {
+ "PATH": "%(PATH)s:RECIPE_PACKAGE[depot_tools]"
+ },
"name": "gclient sync",
"~followup_annotations": [
"@@@STEP_LOG_LINE@json.output@{@@@",
@@ -54,6 +60,9 @@
"local_bot"
],
"cwd": "[SLAVE_BUILD]",
+ "env": {
+ "PATH": "%(PATH)s:RECIPE_PACKAGE[depot_tools]"
+ },
"name": "gclient recurse (git config user.name)"
},
{
@@ -68,6 +77,9 @@
"local_bot@example.com"
],
"cwd": "[SLAVE_BUILD]",
+ "env": {
+ "PATH": "%(PATH)s:RECIPE_PACKAGE[depot_tools]"
+ },
"name": "gclient recurse (git config user.email)"
},
{
@@ -80,6 +92,9 @@
"cache_dir = None\nsolutions = [{'deps_file': 'DEPS', 'managed': True, 'name': 'WebKit', 'safesync_url': 'https://blink-status.appspot.com/lkgr', 'url': 'svn://svn.chromium.org/blink/trunk'}]"
],
"cwd": "[SLAVE_BUILD]/src/third_party",
+ "env": {
+ "PATH": "%(PATH)s:RECIPE_PACKAGE[depot_tools]"
+ },
"name": "[spec: WebKit] gclient setup"
},
{
@@ -99,6 +114,9 @@
"/path/to/tmp/json"
],
"cwd": "[SLAVE_BUILD]/src/third_party",
+ "env": {
+ "PATH": "%(PATH)s:RECIPE_PACKAGE[depot_tools]"
+ },
"name": "[spec: WebKit] gclient sync",
"~followup_annotations": [
"@@@STEP_LOG_LINE@json.output@{@@@",
@@ -147,6 +165,9 @@
"runhooks"
],
"cwd": "[SLAVE_BUILD]",
+ "env": {
+ "PATH": "%(PATH)s:RECIPE_PACKAGE[depot_tools]"
+ },
"name": "gclient runhooks"
},
{

Powered by Google App Engine
This is Rietveld 408576698