Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(77)

Unified Diff: ppapi/ppapi_nacl.gyp

Issue 452543002: Build only the appropriate x86 arches for nacl (chrome side). (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: drop deps Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ppapi/ppapi_nacl.gyp
diff --git a/ppapi/ppapi_nacl.gyp b/ppapi/ppapi_nacl.gyp
index 8252f6d027bc7262a13989609b173abee8c2b16a..2d5684a94fea2b03b6185bace2a0e186a7271369 100644
--- a/ppapi/ppapi_nacl.gyp
+++ b/ppapi/ppapi_nacl.gyp
@@ -98,24 +98,26 @@
'-O0',
],
'conditions': [
- ['target_arch=="ia32" or target_arch=="x64"', {
- 'extra_deps_newlib64': [
- '>(tc_lib_dir_newlib64)/libppapi_cpp.a',
- '>(tc_lib_dir_newlib64)/libppapi.a',
- ],
+ ['target_arch=="ia32"', {
'extra_deps_newlib32': [
'>(tc_lib_dir_newlib32)/libppapi_cpp.a',
'>(tc_lib_dir_newlib32)/libppapi.a',
],
- 'extra_deps_glibc64': [
- '>(tc_lib_dir_glibc64)/libppapi_cpp.so',
- '>(tc_lib_dir_glibc64)/libppapi.so',
- ],
'extra_deps_glibc32': [
'>(tc_lib_dir_glibc32)/libppapi_cpp.so',
'>(tc_lib_dir_glibc32)/libppapi.so',
],
}],
+ ['target_arch=="x64" or (target_arch=="ia32" and OS=="win")', {
+ 'extra_deps_newlib64': [
+ '>(tc_lib_dir_newlib64)/libppapi_cpp.a',
+ '>(tc_lib_dir_newlib64)/libppapi.a',
+ ],
+ 'extra_deps_glibc64': [
+ '>(tc_lib_dir_glibc64)/libppapi_cpp.so',
+ '>(tc_lib_dir_glibc64)/libppapi.so',
+ ],
+ }],
['target_arch=="arm"', {
'extra_deps_arm': [
'>(tc_lib_dir_newlib_arm)/libppapi_cpp.a',
@@ -138,7 +140,7 @@
'create_nonsfi_test_nmf': 'tests/create_nonsfi_test_nmf.py',
},
'conditions': [
- ['target_arch!="arm" and target_arch!="mipsel" and disable_glibc==0', {
+ ['(target_arch=="ia32" or target_arch=="x64") and disable_glibc==0', {
'variables': {
'build_glibc': 1,
# NOTE: Use /lib, not /lib64 here; it is a symbolic link which
@@ -153,7 +155,7 @@
'action_name': 'Generate GLIBC NMF and copy libs',
# NOTE: create_nmf must be first, it is the script python executes
# below.
- 'inputs': ['>(create_nmf)', '>(out_glibc64)', '>(out_glibc32)'],
+ 'inputs': ['>(create_nmf)'],
# NOTE: There is no explicit dependency for the lib32
# and lib64 directories created in the PRODUCT_DIR.
# They are created as a side-effect of NMF creation.
@@ -162,13 +164,25 @@
'python',
'>@(_inputs)',
'--objdump=>(nacl_objdump)',
- '--library-path=>(libdir_glibc64)',
- '--library-path=>(libdir_glibc32)',
- '--library-path=>(tc_lib_dir_glibc32)',
- '--library-path=>(tc_lib_dir_glibc64)',
'--output=>(nmf_glibc)',
'--stage-dependencies=<(PRODUCT_DIR)',
],
+ 'conditions': [
+ ['target_arch=="ia32"', {
+ 'actions': [
+ '--library-path=>(libdir_glibc32)',
+ '--library-path=>(tc_lib_dir_glibc32)',
+ ],
+ 'inputs': ['>(out_glibc32)'],
+ }],
+ ['target_arch=="x64" or (target_arch=="ia32" and OS=="win")', {
+ 'actions': [
+ '--library-path=>(libdir_glibc64)',
+ '--library-path=>(tc_lib_dir_glibc64)',
+ ],
+ 'inputs': ['>(out_glibc64)'],
+ }],
+ ],
},
],
}],
« ppapi/native_client/native_client.gyp ('K') | « ppapi/native_client/native_client.gyp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698