New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 621862 link

Starred by 2 users

Issue metadata

Status: Duplicate
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug

Blocking:
issue 615339



Sign in to add a comment

Make cronet recipes use ensure_goma

Project Member Reported by jbudorick@chromium.org, Jun 21 2016

Issue description

/b/build/goma is going away

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/9b6928a2cef965ed90350f78f62ac2369fc0e813

commit 9b6928a2cef965ed90350f78f62ac2369fc0e813
Author: jbudorick <jbudorick@chromium.org>
Date: Tue Jun 21 14:01:55 2016

[cronet] Switch to ensure_goma.

BUG= 621862 

Review-Url: https://codereview.chromium.org/2083963002

[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipe_modules/cronet/api.py
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipe_modules/cronet/example.expected/gn_test.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipe_modules/cronet/example.expected/local_test.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_arm64_builder.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_arm64_builder__dbg_.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_armv6_builder.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_builder.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_builder__dbg_.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_data_reduction_proxy_builder.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_mips_builder.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_tester.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_x86_builder.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/android_cronet_x86_builder__dbg_.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/cronet_try.json
[modify] https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813/scripts/slave/recipes/cronet.expected/local_test.json

Project Member

Comment 2 by bugdroid1@chromium.org, Jun 21 2016

Project Member

Comment 3 by bugdroid1@chromium.org, Jun 21 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra.git/+/d1bf4408ecfaab3e50188deeec0cafd6342bd28f

commit d1bf4408ecfaab3e50188deeec0cafd6342bd28f
Author: recipe-roller <recipe-roller@chromium.org>
Date: Tue Jun 21 17:21:03 2016

Roll recipe dependencies (trivial).

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).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/9b6928a2cef965ed90350f78f62ac2369fc0e813 [cronet] Switch to ensure_goma. (jbudorick@chromium.org)
  https://crrev.com/daf6b9917c4609610c0a2915b5f5df5a4beeb92c V8: Fix mips toolchain directory (machenbach@chromium.org)

R=jbudorick@chromium.org,jochen@chromium.org,machenbach@chromium.org
BUG= 621862 

TBR=martiniss@chromium.org,phajdan.jr@chromium.org

Review-Url: https://codereview.chromium.org/2087623004

[modify] https://crrev.com/d1bf4408ecfaab3e50188deeec0cafd6342bd28f/infra/config/recipes.cfg

Comment 4 by mef@chromium.org, Jun 21 2016

Thanks for fixing this! 
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 22 2016

Mergedinto: 621818
Status: Duplicate (was: Started)

Sign in to add a comment