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

Issue 623093002: bot_update: Pass custom_vars dict to deps2git. (Closed)

Created:
6 years, 2 months ago by kjellander_chromium
Modified:
6 years, 2 months ago
Reviewers:
hinoka
CC:
chromium-reviews, pgervais+watch_chromium.org, kjellander-cc_chromium.org, cmp-cc_chromium.org, stip+watch_chromium.org
Project:
tools
Visibility:
Public.

Description

bot_update: Pass custom_vars dict to deps2git. To make use of the ability to override the variables in the DEPS file that is supported once https://codereview.chromium.org/619133002/ is landed. BUG=419516

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -0 lines) Patch
M scripts/slave/bot_update.py View 2 chunks +3 lines, -0 lines 1 comment Download

Messages

Total messages: 3 (1 generated)
kjellander_chromium
https://codereview.chromium.org/623093002/diff/1/scripts/slave/bot_update.py File scripts/slave/bot_update.py (right): https://codereview.chromium.org/623093002/diff/1/scripts/slave/bot_update.py#newcode838 scripts/slave/bot_update.py:838: cmd.extend(['--var_overrides', json.dumps(custom_vars)]) I doublechecked that an empty custom_vars dict ...
6 years, 2 months ago (2014-10-03 09:00:50 UTC) #2
kjellander_chromium
6 years, 2 months ago (2014-10-09 09:28:18 UTC) #3
I'm abandoning this CL since I removed the use of the custom_vars "root_dir"
variable that WebRTC had on the bots until now.

Powered by Google App Engine
This is Rietveld 408576698