Use GYP to build nacl_irt

Use GYP to build nacl_irt

Converts Chrome from using scons to using GYP when building nacl_irt.
This change causes chrome to use GYP based components for untrusted
code.  As a benifit, GYP will no longer need to run and scrape scons
output to determine irt dependencies.  This also allows us to avoid
invoking scons during the build process.
We roll NaCl deps to get a fix for a simultaneous 32/64 bit build
issue on Windows.

TEST= try
BUG= http://code.google.com/p/chromium/issues/detail?id=96782
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=108116

[bradn, 2011-10-31 16:38:48]

LGTM

Can we also delete the build_nacl_irt.py script?

http://codereview.chromium.org/8416025/diff/7/chrome/nacl.gypi
File chrome/nacl.gypi (right):

http://codereview.chromium.org/8416025/diff/7/chrome/nacl.gypi#newcode74
chrome/nacl.gypi:74: # shared with the 64-bit target, but it does not work due
to a gyp
Maybe assign that to me...

[noelallen_use_chromium, 2011-10-31 17:51:22]

I'll clean it out after this CL has baked for a while.


On Mon, Oct 31, 2011 at 9:38 AM, <bradnelson@google.com> wrote:

> LGTM
>
> Can we also delete the build_nacl_irt.py script?
>
>
>
http://codereview.chromium.**org/8416025/diff/7/chrome/**nacl.gypi<http://cod...
> File chrome/nacl.gypi (right):
>
> http://codereview.chromium.**org/8416025/diff/7/chrome/**
>
nacl.gypi#newcode74<http://codereview.chromium.org/8416025/diff/7/chrome/nacl.gypi#newcode74>
> chrome/nacl.gypi:74: # shared with the 64-bit target, but it does not
> work due to a gyp
> Maybe assign that to me...
>
>
http://codereview.chromium.**org/8416025/<http://codereview.chromium.org/8416...
>

[bradn, 2011-10-31 17:52:21]

K, lgtm

On Mon, Oct 31, 2011 at 10:51 AM, Noel Allen <noelallen@google.com> wrote:

> I'll clean it out after this CL has baked for a while.
>
>
> On Mon, Oct 31, 2011 at 9:38 AM, <bradnelson@google.com> wrote:
>
>> LGTM
>>
>> Can we also delete the build_nacl_irt.py script?
>>
>>
>>
http://codereview.chromium.**org/8416025/diff/7/chrome/**nacl.gypi<http://cod...
>> File chrome/nacl.gypi (right):
>>
>> http://codereview.chromium.**org/8416025/diff/7/chrome/**
>>
nacl.gypi#newcode74<http://codereview.chromium.org/8416025/diff/7/chrome/nacl.gypi#newcode74>
>> chrome/nacl.gypi:74: # shared with the 64-bit target, but it does not
>> work due to a gyp
>> Maybe assign that to me...
>>
>>
http://codereview.chromium.**org/8416025/<http://codereview.chromium.org/8416...
>>
>
>

[sehr (please use chromium), 2011-10-31 19:47:40]

One nit.  LGTM.

http://codereview.chromium.org/8416025/diff/7/chrome/nacl.gypi
File chrome/nacl.gypi (right):

http://codereview.chromium.org/8416025/diff/7/chrome/nacl.gypi#newcode74
chrome/nacl.gypi:74: # shared with the 64-bit target, but it does not work due
to a gyp
On 2011/10/31 16:38:48, bradn wrote:
> Maybe assign that to me...

Agreed.  The statute of limitations on gregoryd TODOs has probably expired :-)

[dmichael (off chromium), 2011-10-31 19:58:26]

lgtm

rubber stamp

[joth, 2011-11-01 17:14:31]

I think this also broke the linux_redux build.

When you re-land, if you get chance could you also do:
  trychange.py -b linux_redux

Thanks!