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

Issue 2006163004: Make v8_target_arch a GN declare_arg(). (Closed)

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

Description

Make v8_target_arch a GN declare_arg(). There are bots on the chromium.lkgr waterfall that need to be able to set v8_target_arch to "arm" while the rest of the target build is "x86". This CL makes that argument a declare_arg(), so that we can do that. R=machenbach@chromium.org BUG=605732

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -6 lines) Patch
M BUILD.gn View 1 chunk +13 lines, -6 lines 1 comment Download

Messages

Total messages: 6 (1 generated)
Dirk Pranke
4 years, 7 months ago (2016-05-24 01:56:08 UTC) #2
Dirk Pranke
4 years, 7 months ago (2016-05-24 02:16:01 UTC) #3
Michael Achenbach
lgtm with comment https://codereview.chromium.org/2006163004/diff/1/BUILD.gn File BUILD.gn (right): https://codereview.chromium.org/2006163004/diff/1/BUILD.gn#newcode77 BUILD.gn:77: v8_target_arch = "" I've recently added ...
4 years, 7 months ago (2016-05-24 06:39:33 UTC) #4
Michael Achenbach
FYI: Landed this now: https://codereview.chromium.org/2007843002/
4 years, 7 months ago (2016-05-24 18:26:03 UTC) #5
Dirk Pranke
4 years, 7 months ago (2016-05-24 18:41:05 UTC) #6
Message was sent while issue was closed.
On 2016/05/24 18:26:03, Michael Achenbach (slow) wrote:
> FYI: Landed this now: https://codereview.chromium.org/2007843002/

Yup, closing this CL in favor of https://codereview.chromium.org/2007843002/ and
https://codereview.chromium.org/2006213002/ .

Powered by Google App Engine
This is Rietveld 408576698