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

Issue 833563004: Revert of Revert of Add d8 target to the GN build. (Closed)

Created:
5 years, 11 months ago by Yang
Modified:
5 years, 11 months ago
CC:
v8-dev
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

Revert of Revert of Add d8 target to the GN build. (patchset #1 id:1 of https://codereview.chromium.org/838983002/) Reason for revert: Relanding because this is not actually blocking the roll. Original issue's description: > Revert of Add d8 target to the GN build. (patchset #2 id:40001 of https://codereview.chromium.org/834113005/) > > Reason for revert: > V8 roll fails due to this change. (see https://codereview.chromium.org/843673003/) > > Original issue's description: > > Add d8 target to the GN build. > > > > Also formats the rest of the file with "gn format". This accounts for all the changes except for the "d8" target additions. > > > > Committed: https://crrev.com/36383f08c1f692ea797d54a27d7c6b3d3d53b440 > > Cr-Commit-Position: refs/heads/master@{#25984} > > TBR=jochen@chromium.org,machenbach@chromium.org,brettw@chromium.org > NOTREECHECKS=true > NOTRY=true > > Committed: https://crrev.com/ef50fdfdb05e1a3a3822962f5235b804218a5f4e > Cr-Commit-Position: refs/heads/master@{#25985} TBR=jochen@chromium.org,machenbach@chromium.org,brettw@chromium.org NOTREECHECKS=true NOTRY=true Committed: https://crrev.com/ebf3c34fb91b4b61f5c32383a115214c516ab5b0 Cr-Commit-Position: refs/heads/master@{#25996}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+210 lines, -150 lines) Patch
M BUILD.gn View 22 chunks +210 lines, -150 lines 0 comments Download

Messages

Total messages: 12 (1 generated)
Yang
Created Revert of Revert of Add d8 target to the GN build.
5 years, 11 months ago (2015-01-08 13:21:03 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/833563004/1
5 years, 11 months ago (2015-01-08 13:21:50 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 11 months ago (2015-01-08 13:21:58 UTC) #3
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/ebf3c34fb91b4b61f5c32383a115214c516ab5b0 Cr-Commit-Position: refs/heads/master@{#25996}
5 years, 11 months ago (2015-01-08 13:22:13 UTC) #4
brettw
Thanks! I was pretty mystified about the error...
5 years, 11 months ago (2015-01-09 02:24:16 UTC) #5
Michael Achenbach
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/792903005/ by machenbach@chromium.org. ...
5 years, 11 months ago (2015-01-14 11:17:05 UTC) #6
dcarney
A revert of this CL (patchset #1 id:1) has been created in https://codereview.chromium.org/810493004/ by dcarney@chromium.org. ...
5 years, 11 months ago (2015-01-14 11:50:48 UTC) #7
Michael Achenbach
This is odd. The roll right after this revert got through and stayed. Now I ...
5 years, 11 months ago (2015-01-14 21:10:04 UTC) #8
Michael Achenbach
+ dpranke and scottmg regarding the reliability of the win8 gn trybot. How could this ...
5 years, 11 months ago (2015-01-14 21:20:12 UTC) #10
Dirk Pranke
On 2015/01/14 21:20:12, Michael Achenbach wrote: > + dpranke and scottmg regarding the reliability of ...
5 years, 11 months ago (2015-01-14 21:23:07 UTC) #11
Dirk Pranke
5 years, 11 months ago (2015-01-14 21:38:01 UTC) #12
Message was sent while issue was closed.
On 2015/01/14 21:23:07, Dirk Pranke wrote:
> On 2015/01/14 21:20:12, Michael Achenbach wrote:
> > + dpranke and scottmg regarding the reliability of the win8 gn trybot. How
> could
> > this happen:
> > https://codereview.chromium.org/821353005/
> > 
> > Is something wrong with the incremental build?
> 
> It looks like the GN build was broken at tip-of-tree around the same time.
> 
> http://build.chromium.org/p/chromium.win/builders/Win8%20GN?numbuilds=100
>
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_gn_rel
> 
> I think this failure is probably correct.

Here's a bit more info (since I'm not sure exactly what you were asking): the
failure at tip-of-tree looks like it was caused by the v8 autoroll (which you
probably already know).

*That* change -- https://codereview.chromium.org/847343002 -- did not run the
win8 GN bot, because that bot is not currently part of the CQ. So, this sort of
failure was predictable (though rare).

I'm currently in the middle of adding the Mac, Win, and ChromeOS GN bots to the
commit queue -- crbug.com/439402 -- to prevent this problem from happening in
the future.

Powered by Google App Engine
This is Rietveld 408576698