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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2068183002: V8: Move gn-peek step behind mb step (Closed)

Created:
4 years, 6 months ago by Michael Achenbach
Modified:
4 years, 6 months ago
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

V8: Move gn-peek step behind mb step When using mb, gyp is first run in the mb step. Making a comparison with gn makes not much sense before that step. BUG=616035 TBR=kjellander@chromium.org Committed: https://chromium.googlesource.com/chromium/tools/build/+/f34f0324f28fad83526514116bae6b679feaf29a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1541 lines, -1106 lines) Patch
M scripts/slave/recipe_modules/v8/api.py View 1 chunk +1 line, -0 lines 0 comments Download
M scripts/slave/recipes/v8.py View 1 chunk +0 lines, -1 line 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_dart_fyi_v8_linux_release.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_dynamorio_linux_v8_dr.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64_ASAN.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64_ASAN___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64_ASAN_arm64___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64_ASAN_no_inline___release_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64_TSAN.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___cfi.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___custom_snapshot___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___gcov_coverage.json View 2 chunks +20 lines, -20 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___internal_snapshot.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___internal_snapshot_empty_json.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___internal_snapshot_one_failure.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___internal_snapshot_test_failures.json View 4 chunks +103 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___internal_snapshot_test_failures_flakes.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux64___internal_snapshot_test_failures_wrong_results.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux_ASAN_arm___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux_ASAN_mipsel___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___arm64___sim___MSAN.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___full_debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___interpreted_regexp.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___noi18n___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___nosnap_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___nosnap_debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___predictable.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___predictable_bisect.json View 5 chunks +149 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___predictable_bisect_one_change.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___predictable_bisect_override_changes.json View 5 chunks +149 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___predictable_bisect_recurring_failure.json View 3 chunks +57 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___predictable_bisect_tests_too_long.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___presubmit.json View 1 chunk +0 lines, -39 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___shared.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___shared_bisect_swarming.json View 5 chunks +149 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___swarming_staging.json View 2 chunks +33 lines, -33 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux___vtunejit.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_V8_Linux_gcc_4_8.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux64___beta_branch.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux64___beta_branch___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux64___stable_branch.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux64___stable_branch___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux___beta_branch.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux___beta_branch___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux___stable_branch.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_branches_V8_Linux___stable_branch___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___arm64___sim.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___arm64___sim___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___arm64___sim___gc_stress.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___arm64___sim___nosnap___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___arm___sim.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___arm___sim___debug.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___mips64el___sim___builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___mipsel___sim.json View 1 chunk +0 lines, -39 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___mipsel___sim___builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___ppc64___sim.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___ppc___sim.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___s390___sim.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___s390x___sim.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___x87___nosnap___debug.json View 1 chunk +0 lines, -39 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_client_v8_ports_V8_Linux___x87___nosnap___debug_builder.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_asan_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_avx2_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_avx2_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_msan_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_sanitizer_coverage_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux64_tsan_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm64_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm64_gc_stress_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm64_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm_armv8a_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm_armv8a_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm_armv8a_rel_positional_extra_flags.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_arm_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_avx2_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_dbg_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_gc_stress_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_gcc_compile_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_gcc_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_mips64el_compile_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_mipsel_compile_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_nodcheck_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_noi18n_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_nosnap_dbg.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_nosnap_rel.json View 2 chunks +11 lines, -11 lines 0 comments Download
M scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_rel_ng.json View 2 chunks +11 lines, -11 lines 0 comments Download

Dependent Patchsets:

Issue 2070003002 Patch 1
Issue 2070003002 Patch 20001

Messages

Total messages: 10 (5 generated)
Michael Achenbach
PTAL
4 years, 6 months ago (2016-06-15 13:45:45 UTC) #3
Michael Achenbach
TBR to land dependent CL
4 years, 6 months ago (2016-06-15 15:45:44 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2068183002/1
4 years, 6 months ago (2016-06-15 15:48:27 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/tools/build/+/f34f0324f28fad83526514116bae6b679feaf29a
4 years, 6 months ago (2016-06-15 15:52:56 UTC) #9
kjellander_chromium
4 years, 6 months ago (2016-06-15 16:29:54 UTC) #10
Message was sent while issue was closed. 
lgtm

Powered by Google App Engine
This is Rietveld 408576698