Shorten gyp target names for PNaCl IRT Shim

Shorten gyp target names for PNaCl IRT Shim

These names get expanded twice into the pathnames of some built files
which in some cases cause them to exceed Windows' MAX_PATH limit and
kill the build. This reduces those path lengths by ~20 chars.

e.g. ../../../../../out/Release_x64/obj/ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_irt_shim_aot.gen/pnacl_newlib_x86_32-nonsfi/pnacl_irt_shim_aot/irt_shim_ppapi.o.d

becomes
../../../../../out/Release_x64/obj/ppapi/native_client/src/untrusted/pnacl_irt_shim/shim_aot.gen/pnacl_newlib_x86_32-nonsfi/shim_aot/irt_shim_ppapi.o.d

R=binji@chromium.org, dyen@chromium.org, mseaborn@chromium.org, teravest@chromium.org
BUG=https://code.google.com/p/nativeclient/issues/detail?id=3846

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268292

[Mark Seaborn, 2014-05-05 21:02:51]

LGTM, but:

Please can you file a bug describing the problem, and link it in BUG=?  Since
this is just a workaround, the problem still remains.

In the commit message, can you give examples of the full pathnames involved in
the build, to show how trimming 10 chars in the Gyp files translates to 20 chars
in the filenames?

https://codereview.chromium.org/263083012/diff/1/ppapi/native_client/src/untr...
File ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_irt_shim.gyp
(right):

https://codereview.chromium.org/263083012/diff/1/ppapi/native_client/src/untr...
ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_irt_shim.gyp:12:
'target_name': 'shim_aot',
Maybe comment here that we're keeping the name short to avoid Windows
limitations (+ link to bug)?

[David Yen, 2014-05-05 21:06:18]

lgtm

[Derek Schuff, 2014-05-05 21:34:53]

updated commit message

https://codereview.chromium.org/263083012/diff/1/ppapi/native_client/src/untr...
File ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_irt_shim.gyp
(right):

https://codereview.chromium.org/263083012/diff/1/ppapi/native_client/src/untr...
ppapi/native_client/src/untrusted/pnacl_irt_shim/pnacl_irt_shim.gyp:12:
'target_name': 'shim_aot',
On 2014/05/05 21:02:52, Mark Seaborn wrote:
> Maybe comment here that we're keeping the name short to avoid Windows
> limitations (+ link to bug)?

Done.

[Derek Schuff, 2014-05-05 21:36:52]

+binji, teravest for OWNERS in SDK and src/ppapi

[teravest, 2014-05-05 21:38:40]

lgtm

[binji, 2014-05-05 21:57:03]

native_client_sdk lgtm

[Derek Schuff, 2014-05-05 22:06:28]

Committed patchset #2 manually as r268292 (presubmit successful).