ninja: make cross-compilation use $CC/$CXX for the target compiler

ninja: make cross-compilation use $CC/$CXX for the target compiler

BUG=None
TEST=set CC/CXX to cross-compiler, set GYP_CROSSCOMPILE to 1, run gyp, ninja chrome.

[Ami GONE FROM CHROMIUM, 2012-03-09 17:36:45]

LGTM assuming it works :)

(I haven't patched & tried; let me know if you want me to try it on a cros/arm
cross-compile)

[Ami GONE FROM CHROMIUM, 2012-05-10 03:38:25]

Yaron: can you confirm this doesn't break your cross-compile setup?

[Yaron, 2012-05-10 19:08:50]

As is, I think it'll break our setup. See android_gyp in:
http://code.google.com/p/chromium/source/search?q=envsetup.sh&origq=envsetup....
where we only set flags for *_target

That being said, I think we could make it work. Does it work for CrOs?

[Ami GONE FROM CHROMIUM, 2012-05-10 19:12:41]

On 2012/05/10 19:08:50, Yaron wrote:
> As is, I think it'll break our setup. See android_gyp in:
>
http://code.google.com/p/chromium/source/search?q=envsetup.sh&origq=envsetup....
> where we only set flags for *_target

Things should still work b/c *_host fallback is to default command names, not
unadorned env vars.  Can you please try it in your setup?

> That being said, I think we could make it work. Does it work for CrOs?

Yes, tzik@ said it worked for him.

[Yaron, 2012-05-10 20:40:39]

On 2012/05/10 19:12:41, Ami Fischman wrote:
> On 2012/05/10 19:08:50, Yaron wrote:
> > As is, I think it'll break our setup. See android_gyp in:
> >
>
http://code.google.com/p/chromium/source/search?q=envsetup.sh&origq=envsetup....
> > where we only set flags for *_target
> 
> Things should still work b/c *_host fallback is to default command names, not
> unadorned env vars.  Can you please try it in your setup?
> 
Work for us too. Patched in downstream and was able to build. Thanks for looping
me in!

> > That being said, I think we could make it work. Does it work for CrOs?
> 
> Yes, tzik@ said it worked for him.

[michaelbai, 2012-07-26 00:43:20]

Can this be committed? I have a change depend on this.

[Nico, 2012-08-01 18:19:54]

How does this relate to https://chromiumcodereview.appspot.com/10833021/ ?

[Nico, 2012-08-03 04:41:34]

http://codereview.chromium.org/9649016/diff/1/pylib/gyp/generator/ninja.py
File pylib/gyp/generator/ninja.py (right):

http://codereview.chromium.org/9649016/diff/1/pylib/gyp/generator/ninja.py#ne...
pylib/gyp/generator/ninja.py:1069: cc_host, cxx_host = cc, cxx
The mac bots set just 'CC' and nothing else. In that setup, both host and target
compiler need to be set to whatever CC was. With the current patch, that's not
true.

[Ami GONE FROM CHROMIUM, 2012-08-11 17:23:03]

Time to close this in favor of http://code.google.com/p/gyp/source/detail?r=1463
?