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

Issue 1979883002: Reland changes to v8_snapshot GN build arg. (Closed)

Created:
4 years, 7 months ago by Dirk Pranke
Modified:
4 years, 7 months ago
CC:
v8-reviews_googlegroups.com, stevenjb, hashimoto
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

Reland changes to v8_snapshot GN build arg. This patch re-lands #36193 with a couple of minor tweaks including updating the BUILD.gn file to use the new variable name and incorporating thakis' request to support cross-compiling the win snapshot from a mac. R=thakis@chromium.org, jochen@chromium.org, machenbach@chromium.org BUG=608596, 595653 CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:chromeos_daisy_chromium_compile_only_ng Committed: https://crrev.com/b6168e1223d266dba1a357a921bc84ae97a1d803 Cr-Commit-Position: refs/heads/master@{#36342}

Patch Set 1 #

Patch Set 2 : make cros backwards-compatible for now #

Unified diffs Side-by-side diffs Delta from patch set Stats (+40 lines, -15 lines) Patch
M BUILD.gn View 1 4 chunks +4 lines, -4 lines 0 comments Download
M snapshot_toolchain.gni View 1 1 chunk +36 lines, -11 lines 0 comments Download

Messages

Total messages: 26 (12 generated)
Dirk Pranke
I need to think about how to land this CL and https://codereview.chromium.org/1983613002/ to be sure ...
4 years, 7 months ago (2016-05-15 01:36:18 UTC) #1
Dirk Pranke
On 2016/05/15 01:36:18, Dirk Pranke wrote: > I need to think about how to land ...
4 years, 7 months ago (2016-05-16 00:24:21 UTC) #3
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1979883002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1979883002/1
4 years, 7 months ago (2016-05-16 00:24:31 UTC) #5
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 7 months ago (2016-05-16 00:52:57 UTC) #7
stevenjb
Not an owner but I tested this with https://codereview.chromium.org/1983613002/ and https://chromium-review.googlesource.com/#/c/344912/. LGTM
4 years, 7 months ago (2016-05-16 23:57:39 UTC) #9
adamk
lgtm as long as you wait for the Chromium side to land first (naturally).
4 years, 7 months ago (2016-05-17 00:08:15 UTC) #11
Michael Achenbach
lgtm - note that we can use any chromium trybot on v8 patches to preview ...
4 years, 7 months ago (2016-05-17 08:02:25 UTC) #13
Dirk Pranke
On 2016/05/17 08:02:25, Michael Achenbach (slow) wrote: > lgtm - note that we can use ...
4 years, 7 months ago (2016-05-17 20:09:55 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1979883002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1979883002/1
4 years, 7 months ago (2016-05-18 21:58:33 UTC) #16
commit-bot: I haz the power
Try jobs failed on following builders: chromeos_daisy_chromium_compile_only_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_chromium_compile_only_ng/builds/139836)
4 years, 7 months ago (2016-05-18 22:07:16 UTC) #18
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1979883002/20001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1979883002/20001
4 years, 7 months ago (2016-05-19 00:21:58 UTC) #21
commit-bot: I haz the power
Committed patchset #2 (id:20001)
4 years, 7 months ago (2016-05-19 01:04:24 UTC) #23
commit-bot: I haz the power
Patchset 2 (id:??) landed as https://crrev.com/b6168e1223d266dba1a357a921bc84ae97a1d803 Cr-Commit-Position: refs/heads/master@{#36342}
4 years, 7 months ago (2016-05-19 01:06:46 UTC) #25
Michael Achenbach
4 years, 7 months ago (2016-05-19 07:10:07 UTC) #26
Message was sent while issue was closed.
A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/1996533002/ by machenbach@chromium.org.

The reason for reverting is: Blocks the roll again:
https://codereview.chromium.org/1993163002/

https://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64...

Lets add chromeos_amd64-generic_chromium_compile_only_ng on a reland as well..

Powered by Google App Engine
This is Rietveld 408576698