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

Issue 2104483003: Roll recipe dependencies (nontrivial). (Closed)

Created:
4 years, 5 months ago by recipe-roller
Modified:
4 years, 5 months ago
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/tools/build@master
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Roll recipe dependencies (nontrivial). This is an automated CL created by the recipe roller. This CL rolls recipe changes from upstream projects (e.g. depot_tools) into downstream projects (e.g. tools/build). Please review the expectation changes, and LGTM as normal. The recipe roller will *NOT* CQ the change itself, so you must commit the change manually. More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug (or complain) depot_tools: https://crrev.com/460d557eecd35efcacb1a0d04b5c7a0ad0fcd84d Revert of infra_paths: keep cache paths super short to avoid long path issues (patchset #1 id:1 of https://codereview.chromium.org/2102613002/ ) (nodir@chromium.org) R=tandrii@chromium.org,emso@chromium.org,thakis@chromium.org,nodir@chromium.org,phajdan.jr@chromium.org,sergiyb@chromium.org BUG=623575 Committed: https://chromium.googlesource.com/chromium/tools/build/+/5934ca1eb6587a02ea079745db921e829d54c031

Patch Set 1 #

Patch Set 2 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M infra/config/recipes.cfg View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 22 (9 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2104483003/1
4 years, 5 months ago (2016-06-27 21:16:37 UTC) #2
recipe-roller
4 years, 5 months ago (2016-06-27 21:16:42 UTC) #3
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 5 months ago (2016-06-27 21:20:57 UTC) #5
recipe-roller
Rebase
4 years, 5 months ago (2016-06-27 21:22:38 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2104483003/20001
4 years, 5 months ago (2016-06-27 21:22:44 UTC) #8
commit-bot: I haz the power
No L-G-T-M from a valid reviewer yet. CQ run can only be started by full ...
4 years, 5 months ago (2016-06-27 21:22:46 UTC) #10
nodir
lgtm
4 years, 5 months ago (2016-06-27 21:26:44 UTC) #12
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2104483003/20001
4 years, 5 months ago (2016-06-27 21:26:54 UTC) #13
commit-bot: I haz the power
Try jobs failed on following builders: Build Presubmit on master.tryserver.infra (JOB_FAILED, https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/builds/5766)
4 years, 5 months ago (2016-06-27 21:34:56 UTC) #15
nodir
PTAL
4 years, 5 months ago (2016-06-27 21:49:14 UTC) #17
estaab
lgtm
4 years, 5 months ago (2016-06-27 21:58:53 UTC) #18
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2104483003/20001
4 years, 5 months ago (2016-06-27 22:13:30 UTC) #20
commit-bot: I haz the power
4 years, 5 months ago (2016-06-27 22:17:14 UTC) #22
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/tools/build/+/5934ca1eb6587a02ea07...

Powered by Google App Engine
This is Rietveld 408576698