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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2342863002: Revert of 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

Revert of Enable the ninja up-to-date check for Android builders (patchset #1 id:1 of https://codereview.chromium.org/2343563003/ ) Reason for revert: Broke cronet: https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20MIPS%20Builder/builds/7317/steps/compile/logs/stdio ninja explain: restat of output md5sum_bin_host older than most recent input obj/tools/android/md5sum/md5sum_bin_host.inputdeps.stamp (0 vs 1473949042) ninja explain: md5sum_bin_host is dirty ninja explain: obj/tools/android/md5sum/md5sum_bin_host.stamp is dirty ninja explain: restat of output host_forwarder older than most recent input obj/tools/android/forwarder2/host_forwarder.inputdeps.stamp (0 vs 1473949042) ninja explain: host_forwarder is dirty Original issue's description: > Reland of Enable the ninja up-to-date check for Android builders > > Reverted in: > https://codereview.chromium.org/2340093002/ > > Reason for reland: > Fixed cronet targets here: > https://codereview.chromium.org/2341903002 > > BUG=646165 > > Committed: https://chromium.googlesource.com/chromium/tools/build/+/f0f4c4a27746dee8a53234f03e452dba556deb55 TBR=dpranke@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=646165 Committed: https://chromium.googlesource.com/chromium/tools/build/+/2744734cf3aa1d8a02d0f0b7abf99635e9f2aa48

Patch Set 1 #

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

Messages

Total messages: 7 (2 generated)
agrieve
Created Revert of Enable the ninja up-to-date check for Android builders
4 years, 3 months ago (2016-09-15 14:22:11 UTC) #2
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/2342863002/1
4 years, 3 months ago (2016-09-15 14:22:18 UTC) #3
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/tools/build/+/2744734cf3aa1d8a02d0f0b7abf99635e9f2aa48
4 years, 3 months ago (2016-09-15 14:22:39 UTC) #5
agrieve
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/2343953002/ by agrieve@chromium.org. ...
4 years, 3 months ago (2016-09-15 18:00:15 UTC) #6
Dirk Pranke
4 years, 3 months ago (2016-09-15 19:31:50 UTC) #7
Message was sent while issue was closed. 
rs- lgtm.

Powered by Google App Engine
This is Rietveld 408576698