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

Unified Diff: ppapi/native_client/src/shared/ppapi/nacl.scons

Issue 8632023: Avoid references to untrusted.gypi in WebKit (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years, 1 month 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 | « chrome/chrome_tests.gypi ('k') | ppapi/ppapi.gyp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/native_client/src/shared/ppapi/nacl.scons
===================================================================
--- ppapi/native_client/src/shared/ppapi/nacl.scons (revision 111604)
+++ ppapi/native_client/src/shared/ppapi/nacl.scons (working copy)
@@ -16,17 +16,24 @@
env.FilterOut(CCFLAGS=['-Wswitch-enum'])
# Load ppapi_cpp.gypi
+# Unfortunately gyp_extract does not handle variables or includes so we must
+# pull the list of sources from ppapi_sources.gypi directly.
+
+ppapi_sources_gypi = open(
+ env.File('$SOURCE_ROOT/ppapi/ppapi_sources.gypi').abspath, 'r').read()
+ppapi_sources_list = eval(ppapi_sources_gypi)['variables']['cpp_sources']
+
+# From ppapi_cpp.gypi:ppapi_cpp:.*\.cc
ppapi_cpp_gypi = gyp_extract.LoadGypFile(
env.File('$SOURCE_ROOT/ppapi/ppapi_cpp.gypi').abspath)
-# From ppapi_cpp.gypi:ppapi_cpp_objects:.*\.cc
-# From ppapi_cpp.gypi:ppapi_cpp:.*\.cc
+
cpp_sources = (
+ [src for src in ppapi_sources_list if src.endswith('.cc')] +
gyp_extract.GypTargetSources(
- ppapi_cpp_gypi, 'ppapi_cpp_objects', '.*\.cc') +
- gyp_extract.GypTargetSources(
ppapi_cpp_gypi, 'ppapi_cpp', '.*\.cc')
)
+
libppapi_cpp = env.NaClSdkLibrary('libppapi_cpp', cpp_sources)
env.AddLibraryToSdk(libppapi_cpp)
« no previous file with comments | « chrome/chrome_tests.gypi ('k') | ppapi/ppapi.gyp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698