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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2338203004: Enable the ninja up-to-date check for Android builders (Closed)

Created:
4 years, 3 months ago by agrieve
Modified:
4 years, 3 months ago
Reviewers:
Dirk Pranke
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Enable the ninja up-to-date check for Android builders BUG=646165 Committed: https://chromium.googlesource.com/chromium/tools/build/+/9ef12eb44ae227823398dc0604213fadae4e1655

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+76 lines, -1 line) Patch
M scripts/slave/recipe_modules/chromium/example.expected/basic_out_dir.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipe_modules/chromium_android/chromium_config.py View 1 chunk +2 lines, -1 line 0 comments Download
M scripts/slave/recipe_modules/chromium_android/example.expected/basic_builder_basic.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipe_modules/chromium_android/example.expected/coverage_builder_basic.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipe_modules/chromium_android/example.expected/gerrit_refs.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipe_modules/chromium_android/example.expected/gerrit_try_builder_basic.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipe_modules/chromium_android/example.expected/restart_usb_builder_basic.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipe_modules/cronet/example.expected/gn_test.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipe_modules/cronet/example.expected/local_test.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/chromium_android_Android_x64_Builder__dbg__fail_check_licenses.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_chromium_android_Android_MIPS_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_chromium_android_Android_x64_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_chromium_android_Android_x86_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_chromium_perf_Android_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_chromium_perf_Android_arm64_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_client_v8_fyi_Android_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_tryserver_chromium_perf_android_arm64_perf_bisect_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/android/builder.expected/full_tryserver_chromium_perf_android_perf_bisect_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/dynamic_instrumentation_nodefault_build.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/dynamic_swarmed_gn_instrumentation_test.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_Android.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_Cronet_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_MIPS_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_N5X_Swarm_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_Swarm_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_arm64_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_arm_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_x64_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_android_Android_x86_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Asan_Builder_Tests__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_fyi_Android_Cloud_Tests.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_gpu_fyi_Android_Release__Nexus_5X_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_gpu_fyi_Android_Release__Nexus_5_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_gpu_fyi_Android_Release__Nexus_6P_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_gpu_fyi_Android_Release__Nexus_6_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_gpu_fyi_Android_Release__Nexus_9_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_gpu_fyi_Android_Release__Pixel_C_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Android_Arm64_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Android_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Android_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Android_Clang_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_linux_Cast_Android__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_webkit_Android_Builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_webrtc_fyi_Android_Builder_ARM64__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_chromium_webrtc_fyi_Android_Builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium.expected/full_client_v8_fyi_V8_Android_GN__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium_trybot.expected/full_tryserver_chromium_android_android_archive_rel_ng.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium_trybot.expected/full_tryserver_chromium_android_android_blink_rel.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium_trybot.expected/full_tryserver_chromium_android_android_coverage.json View 2 chunks +2 lines, -0 lines 0 comments Download
M scripts/slave/recipes/chromium_trybot.expected/full_tryserver_chromium_android_cast_shell_android.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/chromium_trybot.expected/full_tryserver_v8_v8_android_chromium_gn_dbg.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_arm64_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_arm64_builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_armv6_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_builder_asan.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_data_reduction_proxy_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_kitkat_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_lollipop_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_marshmallow_64bit_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_marshmallow_64bit_perf.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_mips_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_tester.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_x86_builder.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/android_cronet_x86_builder__dbg_.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/cronet_try.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/cronet.expected/local_test.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/dart/dartium_android_builder.expected/dartium_builder_basic.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/dart/dartium_android_builder.expected/dartium_builder_git.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/emulator.expected/Android_Tests__x86_emulator__test_basic.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/emulator.expected/Android_Tests__x86_emulator__test_fail.json View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/swarming/deterministic_build.expected/full_chromium_swarm_Android_deterministic.json View 2 chunks +2 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
agrieve
On 2016/09/14 14:29:56, agrieve wrote: > mailto:agrieve@chromium.org changed reviewers: > + mailto:dpranke@chromium.org
4 years, 3 months ago (2016-09-14 14:30:03 UTC) #3
Dirk Pranke
lgtm
4 years, 3 months ago (2016-09-14 17:28:59 UTC) #4
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/2338203004/1
4 years, 3 months ago (2016-09-14 17:59:30 UTC) #6
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/tools/build/+/9ef12eb44ae227823398dc0604213fadae4e1655
4 years, 3 months ago (2016-09-14 18:06:27 UTC) #8
agrieve
4 years, 3 months ago (2016-09-14 18:58:22 UTC) #9
Message was sent while issue was closed. 
A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2340093002/ by agrieve@chromium.org.

The reason for reverting is: Broke cronet builders.
.

Powered by Google App Engine
This is Rietveld 408576698