Build only the appropriate x86 arches for nacl (chrome side).

Build only the appropriate x86 arches for nacl (chrome side).

BUG=https://code.google.com/p/chromium/issues/detail?id=388101
TEST=local + with trybots
R=mseaborn@chromium.org,jam@chromium.org,brettw@chromium.org

Committed: https://crrev.com/5c5b64142652bfa57a4309602892765b889350dc
Cr-Commit-Position: refs/heads/master@{#294987}

[Mark Seaborn, 2014-08-07 19:30:47]

LGTM, but please test this via a clobber build before committing.

https://codereview.chromium.org/452543002/diff/20001/ppapi/native_client/nati...
File ppapi/native_client/native_client.gyp (right):

https://codereview.chromium.org/452543002/diff/20001/ppapi/native_client/nati...
ppapi/native_client/native_client.gyp:69: ['target_arch=="x64" or
(target_arch=="ia32" and OS=="win")', {
I have a similar comment to https://codereview.chromium.org/366763003/:

To avoid duplication of this check, can we define some "build for x86_32" and
"build for x86_64" variables in a single place?

[Mark Seaborn, 2014-08-07 19:31:59]

I spoke too soon.  It looks like this fails on the trybots:

create_nmf.py: cannot find library libpthread.so.9b15f6a6

(from
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[jam, 2014-08-08 01:27:00]

lgtm, thanks

[bradn, 2014-09-16 00:01:17]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2014-09-16 00:37:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/452543002/160001

[commit-bot: I haz the power, 2014-09-16 01:18:22]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-09-16 01:18:23]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[bbudge, 2014-09-16 02:14:54]

bbudge@chromium.org changed reviewers:
+ bbudge@chromium.org

[bbudge, 2014-09-16 02:14:55]

Rubberstamp lgtm

[bradn, 2014-09-16 02:16:25]

The CQ bit was checked by bradnelson@google.com

[brettw, 2014-09-16 02:17:52]

rs lgtm

[commit-bot: I haz the power, 2014-09-16 02:19:13]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/452543002/160001

[commit-bot: I haz the power, 2014-09-16 02:31:23]

Committed patchset #9 (id:160001) as 9386fe3d8d3d6fd481d09eb445e3827cba5d76bb

[commit-bot: I haz the power, 2014-09-16 02:34:43]

Patchset 9 (id:??) landed as
https://crrev.com/5c5b64142652bfa57a4309602892765b889350dc
Cr-Commit-Position: refs/heads/master@{#294987}