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

Unified Diff: ppapi/ppapi_untrusted.gyp

Issue 10409032: Enable PPAPINaCl GLIBC browser_tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: More ARM build fixes Created 8 years, 6 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
« no previous file with comments | « ppapi/native_client/tests/ppapi_tests/ppapi_nacl_tests_newlib.nmf ('k') | ppapi/tests/test_case.html » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/ppapi_untrusted.gyp
diff --git a/ppapi/ppapi_untrusted.gyp b/ppapi/ppapi_untrusted.gyp
index 57fe4a1f1206c0d5fe0cb7073e2ce2b006383bd6..db686e4b76fffd39287e60820b4cf05c4a3867ae 100644
--- a/ppapi/ppapi_untrusted.gyp
+++ b/ppapi/ppapi_untrusted.gyp
@@ -1,4 +1,4 @@
-# Copyright (c) 2011 The Chromium Authors. All rights reserved.
+# Copyright (c) 2012 The Chromium Authors. All rights reserved.
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
@@ -17,7 +17,7 @@
'type': 'none',
'variables': {
'nlib_target': 'libppapi_cpp.a',
- 'build_glibc': 0,
+ 'build_glibc': 1,
'build_newlib': 1,
'sources': [
'<@(cpp_source_files)',
@@ -39,7 +39,6 @@
],
'variables': {
'nexe_target': 'ppapi_nacl_tests',
- 'build_glibc': 0,
'build_newlib': 1,
'include_dirs': [
'lib/gl/include',
@@ -55,10 +54,6 @@
'<(SHARED_INTERMEDIATE_DIR)/tc_newlib/lib64/libppapi_cpp.a',
'<(SHARED_INTERMEDIATE_DIR)/tc_newlib/lib64/libppapi.a',
],
- 'extra_deps32': [
- '<(SHARED_INTERMEDIATE_DIR)/tc_newlib/lib32/libppapi_cpp.a',
- '<(SHARED_INTERMEDIATE_DIR)/tc_newlib/lib32/libppapi.a',
- ],
'extra_deps_newlib64': [
'<(SHARED_INTERMEDIATE_DIR)/tc_newlib/lib64/libppapi_cpp.a',
'<(SHARED_INTERMEDIATE_DIR)/tc_newlib/lib64/libppapi.a',
@@ -84,6 +79,39 @@
'<@(test_nacl_source_files)',
],
},
+ 'conditions': [
+ ['target_arch!="arm"', {
+ 'variables': {
+ 'build_glibc': 1,
+ # NOTE: Use /lib, not /lib64 here; it is a symbolic link which
+ # doesn't work on Windows.
+ 'libdir_glibc64': '>(nacl_glibc_tc_root)/x86_64-nacl/lib',
+ 'libdir_glibc32': '>(nacl_glibc_tc_root)/x86_64-nacl/lib32',
+ 'nacl_objdump': '>(nacl_glibc_tc_root)/bin/x86_64-nacl-objdump',
+ 'nmf_glibc%': '<(PRODUCT_DIR)/>(nexe_target)_glibc.nmf',
+ },
+ 'actions': [
+ {
+ 'action_name': 'Generate GLIBC NMF and copy libs',
+ 'inputs': ['>(out_glibc64)', '>(out_glibc32)'],
+ # 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.
+ 'outputs': ['>(nmf_glibc)'],
+ 'action': [
+ 'python',
+ '<(DEPTH)/native_client_sdk/src/tools/create_nmf.py',
+ '>@(_inputs)',
+ '--objdump=>(nacl_objdump)',
+ '--library-path=>(libdir_glibc64)',
+ '--library-path=>(libdir_glibc32)',
+ '--output=>(nmf_glibc)',
+ '--stage-dependencies=<(PRODUCT_DIR)',
+ ],
+ },
+ ],
+ }],
+ ],
},
],
}
« no previous file with comments | « ppapi/native_client/tests/ppapi_tests/ppapi_nacl_tests_newlib.nmf ('k') | ppapi/tests/test_case.html » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698