Chromium Code Reviews
Help | Chromium Project | Sign in
(370)

Issue 9417030: Enable cross-compilation with ninja. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
2 years, 2 months ago by Ami Fischman
Modified:
2 years, 1 month ago
Reviewers:
Evan Martin
CC:
gyp-developer_googlegroups.com, piman
Base URL:
http://git.chromium.org/external/gyp.git@master
Visibility:
Public.

Description

Enable cross-compilation with ninja.

Replaced the specialness of _host with _target, b/c the _host
toolchain is usually the vanilla gcc/g++, so it looks better to
leave those variables alone (plus, it mirrors make.py and
GypPathToUniqueOutput).

Added an AR to the family b/c it differs between host & target.

Made generator_supports_multiple_toolsets depend on the presence
of any of the $FOO_target variables as an intermediate hack
around the double-building problem.

BUG=gyp:213
Committed: https://code.google.com/p/gyp/source/detail?r=1231

Patch Set 1 #

Patch Set 2 : Default target tools to host tools #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -11 lines) Lint Patch
M pylib/gyp/generator/ninja.py View 1 4 chunks +21 lines, -11 lines 1 comment ? errors Download
Trybot results:
Commit:

Messages

Total messages: 7
Ami Fischman
Evan: please review. Antoine: FYI. I replaced the specialness of _host with _target, b/c the ...
2 years, 2 months ago #1
Evan Martin
On 2012/02/16 23:43:26, Ami Fischman wrote: > Evan: please review. > Antoine: FYI. > > ...
2 years, 2 months ago #2
Ami Fischman
On 2012/02/16 23:48:32, Evan Martin wrote: > Can you put some of these comments in ...
2 years, 2 months ago #3
Ami Fischman
Evan: ping?
2 years, 2 months ago #4
Evan Martin
LGTM
2 years, 2 months ago #5
Ami Fischman
Evan: PTAL & land if LGTY. https://chromiumcodereview.appspot.com/9417030/diff/5001/pylib/gyp/generator/ninja.py File pylib/gyp/generator/ninja.py (right): https://chromiumcodereview.appspot.com/9417030/diff/5001/pylib/gyp/generator/ninja.py#newcode1075 pylib/gyp/generator/ninja.py:1075: master_ninja.variable('ar_target', os.environ.get('AR_target', '$ar')) ...
2 years, 1 month ago #6
Evan Martin
2 years, 1 month ago #7
r213
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6