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

Issue 1660943002: bot_update: add depot_tools to PATH so that e.g. apply_issue can be found (Closed)

Created:
4 years, 10 months ago by Paweł Hajdan Jr.
Modified:
4 years, 10 months ago
CC:
chromium-reviews, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

bot_update: add depot_tools to PATH so that e.g. apply_issue can be found BUG=437264 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=298556

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -0 lines) Patch
M recipe_modules/bot_update/api.py View 1 chunk +4 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/basic.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/basic_output_manifest.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/basic_with_branch_heads.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/clobber.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/forced.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/no_shallow.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/off.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/reset_root_solution_revision.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/svn_mode.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/trychange.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/trychange_oauth2.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/tryjob.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/tryjob_fail.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/tryjob_fail_patch.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/tryjob_fail_patch_download.json View 1 chunk +3 lines, -0 lines 0 comments Download
M recipe_modules/bot_update/example.expected/tryjob_v8.json View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 12 (5 generated)
Paweł Hajdan Jr.
This fixes the error from https://chromium-swarm.appspot.com/user/task/2cc02cf5cd9eb710 : ===Running apply_issue --root_dir build --issue 1597473002 --server codereview.chromium.org ...
4 years, 10 months ago (2016-02-03 14:26:58 UTC) #2
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1660943002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1660943002/1
4 years, 10 months ago (2016-02-03 14:27:12 UTC) #4
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 10 months ago (2016-02-03 14:29:11 UTC) #6
jochen (gone - plz use gerrit)
lgtm
4 years, 10 months ago (2016-02-03 14:48:36 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1660943002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1660943002/1
4 years, 10 months ago (2016-02-03 14:49:33 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1) as http://src.chromium.org/viewvc/chrome?view=rev&revision=298556
4 years, 10 months ago (2016-02-03 14:51:36 UTC) #11
kjellander_chromium
4 years, 10 months ago (2016-02-03 15:33:46 UTC) #12
Message was sent while issue was closed.
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1662763003/ by kjellander@chromium.org.

The reason for reverting is: I believe this breaks trybot patching on Windows. 

Example:
https://build.chromium.org/p/tryserver.webrtc/builders/win_x64_gn_rel/builds/...

Error: 
'E:\b\build\scripts\slave\.recipe_deps\depot_tools\python' is not recognized as
an internal or external command,
operable program or batch file.

I'm speculatively reverting..

Powered by Google App Engine
This is Rietveld 408576698