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

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

Issue 6715019: Switching ppapi on nacl side to import chrome side .gyp[i] directly. (Closed) Base URL: svn://svn.chromium.org/native_client/trunk/src/native_client/
Patch Set: '' Created 9 years, 9 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: src/shared/ppapi/nacl.scons
===================================================================
--- src/shared/ppapi/nacl.scons (revision 4588)
+++ src/shared/ppapi/nacl.scons (working copy)
@@ -28,224 +28,75 @@
# two markers.
import os
+import re
Import('env')
# Underlay $SOURCE_ROOT/ppapi in this directory.
Dir('.').addRepository(Dir('#/../ppapi'))
-cpp_sources = [
- # Updated automatically by update-scons.py.
- # From ppapi_cpp.gypi:ppapi_cpp_objects:.*\.cc
- 'cpp/audio.cc',
- 'cpp/audio_config.cc',
- 'cpp/core.cc',
- 'cpp/graphics_2d.cc',
- 'cpp/image_data.cc',
- 'cpp/instance.cc',
- 'cpp/module.cc',
- 'cpp/paint_aggregator.cc',
- 'cpp/paint_manager.cc',
- 'cpp/rect.cc',
- 'cpp/resource.cc',
- 'cpp/url_loader.cc',
- 'cpp/url_request_info.cc',
- 'cpp/url_response_info.cc',
- 'cpp/var.cc',
- 'cpp/dev/buffer_dev.cc',
- 'cpp/dev/context_3d_dev.cc',
- 'cpp/dev/directory_entry_dev.cc',
- 'cpp/dev/directory_reader_dev.cc',
- 'cpp/dev/file_chooser_dev.cc',
- 'cpp/dev/file_io_dev.cc',
- 'cpp/dev/file_ref_dev.cc',
- 'cpp/dev/file_system_dev.cc',
- 'cpp/dev/find_dev.cc',
- 'cpp/dev/font_dev.cc',
- 'cpp/dev/fullscreen_dev.cc',
- 'cpp/dev/graphics_3d_client_dev.cc',
- 'cpp/dev/graphics_3d_dev.cc',
- 'cpp/dev/printing_dev.cc',
- 'cpp/dev/scrollbar_dev.cc',
- 'cpp/dev/selection_dev.cc',
- 'cpp/dev/surface_3d_dev.cc',
- 'cpp/dev/transport_dev.cc',
- 'cpp/dev/url_util_dev.cc',
- 'cpp/dev/video_decoder_dev.cc',
- 'cpp/dev/widget_client_dev.cc',
- 'cpp/dev/widget_dev.cc',
- 'cpp/dev/zoom_dev.cc',
- 'cpp/dev/scriptable_object_deprecated.cc',
- # End ppapi_cpp.gypi
- # Updated automatically by update-scons.py.
- # From ppapi_cpp.gypi:ppapi_cpp:.*\.cc
- 'cpp/ppp_entrypoints.cc',
- # End ppapi_cpp.gypi
- ]
+# Load ppapi_cpp.gypi
+ppapi_cpp_gypi_filename = env.File('#/../ppapi/ppapi_cpp.gypi').path
+ppapi_cpp_gypi = eval(open(ppapi_cpp_gypi_filename).read())
Mark Seaborn 2011/03/22 01:17:27 Ditto. Hmm, all this logic is duplicated.
bradn 2011/03/22 01:51:43 Done.
+
+# Extract several targets from ppapi_cpp.gypi
+ppapi_cpp_objects = [t for t in ppapi_cpp_gypi['targets']
+ if t['target_name'] == 'ppapi_cpp_objects'][0]
+ppapi_cpp = [t for t in ppapi_cpp_gypi['targets']
+ if t['target_name'] == 'ppapi_cpp'][0]
+ppapi_c = [t for t in ppapi_cpp_gypi['targets']
+ if t['target_name'] == 'ppapi_c'][0]
+
+# From ppapi_cpp.gypi:ppapi_cpp_objects:.*\.cc
+# From ppapi_cpp.gypi:ppapi_cpp:.*\.cc
+cpp_sources = ([s for s in ppapi_cpp_objects['sources']
+ if s.endswith('.cc')] +
+ [s for s in ppapi_cpp['sources']
+ if s.endswith('.cc')])
env.ComponentLibrary('ppapi_cpp', cpp_sources)
+
env.Append(CPPPATH=[
'$SOURCE_ROOT/ppapi/lib/gl/gles2'])
-gles2_sources = [
- # Updated automatically by update-scons.py.
- # From ppapi_gl.gypi:ppapi_gles2:.*\.c
- 'lib/gl/gles2/gl2ext_ppapi.c',
- 'lib/gl/gles2/gles2.c',
- # End ppapi_gl.gypi
-]
+# Load ppapi_gl.gypi
+ppapi_cpp_gypi_filename = env.File('#/../ppapi/ppapi_gl.gypi').path
+ppapi_cpp_gypi = eval(open(ppapi_cpp_gypi_filename).read())
+# Extract ppapi_cpp + ppapi_cpp_objects.
+ppapi_gles2 = [t for t in ppapi_cpp_gypi['targets']
+ if t['target_name'] == 'ppapi_gles2'][0]
+# From ppapi_gl.gypi:ppapi_gles2:.*\.c
+gles2_sources = [s for s in ppapi_gles2['sources']
+ if s.endswith('.cpp')]
env.ComponentLibrary('ppapi_gles2', gles2_sources)
-c_headers = [
- # Updated automatically by update-scons.py.
- # From ppapi_cpp.gypi:ppapi_c:c/[^/]*\.h
- 'c/pp_bool.h',
- 'c/pp_completion_callback.h',
- 'c/pp_errors.h',
- 'c/pp_input_event.h',
- 'c/pp_instance.h',
- 'c/pp_macros.h',
- 'c/pp_module.h',
- 'c/pp_point.h',
- 'c/pp_rect.h',
- 'c/pp_resource.h',
- 'c/pp_size.h',
- 'c/pp_stdint.h',
- 'c/pp_time.h',
- 'c/pp_var.h',
- 'c/ppb.h',
- 'c/ppb_audio.h',
- 'c/ppb_audio_config.h',
- 'c/ppb_core.h',
- 'c/ppb_class.h',
- 'c/ppb_graphics_2d.h',
- 'c/ppb_image_data.h',
- 'c/ppb_instance.h',
- 'c/ppb_url_loader.h',
- 'c/ppb_url_request_info.h',
- 'c/ppb_url_response_info.h',
- 'c/ppb_var.h',
- 'c/ppp.h',
- 'c/ppp_instance.h',
- # End ppapi_cpp.gypi
- ]
-c_dev_headers = [
- # Updated automatically by update-scons.py.
- # From ppapi_cpp.gypi:ppapi_c:c/dev/[^/]*\.h
- 'c/dev/pp_cursor_type_dev.h',
- 'c/dev/pp_file_info_dev.h',
- 'c/dev/pp_graphics_3d_dev.h',
- 'c/dev/pp_video_dev.h',
- 'c/dev/ppb_buffer_dev.h',
- 'c/dev/ppb_char_set_dev.h',
- 'c/dev/ppb_context_3d_dev.h',
- 'c/dev/ppb_context_3d_trusted_dev.h',
- 'c/dev/ppb_cursor_control_dev.h',
- 'c/dev/ppb_directory_reader_dev.h',
- 'c/dev/ppb_file_chooser_dev.h',
- 'c/dev/ppb_file_io_dev.h',
- 'c/dev/ppb_file_io_trusted_dev.h',
- 'c/dev/ppb_file_ref_dev.h',
- 'c/dev/ppb_file_system_dev.h',
- 'c/dev/ppb_find_dev.h',
- 'c/dev/ppb_font_dev.h',
- 'c/dev/ppb_fullscreen_dev.h',
- 'c/dev/ppb_graphics_3d_dev.h',
- 'c/dev/ppb_opengles_dev.h',
- 'c/dev/ppb_scrollbar_dev.h',
- 'c/dev/ppb_surface_3d_dev.h',
- 'c/dev/ppb_testing_dev.h',
- 'c/dev/ppb_transport_dev.h',
- 'c/dev/ppb_url_util_dev.h',
- 'c/dev/ppb_video_decoder_dev.h',
- 'c/dev/ppb_widget_dev.h',
- 'c/dev/ppb_zoom_dev.h',
- 'c/dev/ppp_cursor_control_dev.h',
- 'c/dev/ppp_find_dev.h',
- 'c/dev/ppp_graphics_3d_dev.h',
- 'c/dev/ppp_scrollbar_dev.h',
- 'c/dev/ppp_selection_dev.h',
- 'c/dev/ppp_printing_dev.h',
- 'c/dev/ppp_widget_dev.h',
- 'c/dev/ppp_zoom_dev.h',
- 'c/dev/deprecated_bool.h',
- 'c/dev/ppb_var_deprecated.h',
- 'c/dev/ppp_class_deprecated.h',
- # End ppapi_cpp.gypi
- ]
+# From ppapi_cpp.gypi:ppapi_c:c/[^/]*\.h
+c_headers = [s for s in ppapi_c['sources']
+ if re.match('^c/[^/]*\.h#', s)]
-cpp_headers = [
- # Updated automatically by update-scons.py.
- # From ppapi_cpp.gypi:ppapi_cpp_objects:cpp/[^/]*\.h
- 'cpp/audio.h',
- 'cpp/audio_config.h',
- 'cpp/common.h',
- 'cpp/completion_callback.h',
- 'cpp/core.h',
- 'cpp/graphics_2d.h',
- 'cpp/image_data.h',
- 'cpp/instance.h',
- 'cpp/logging.h',
- 'cpp/module.h',
- 'cpp/module_impl.h',
- 'cpp/non_thread_safe_ref_count.h',
- 'cpp/paint_aggregator.h',
- 'cpp/paint_manager.h',
- 'cpp/point.h',
- 'cpp/rect.h',
- 'cpp/resource.h',
- 'cpp/size.h',
- 'cpp/url_loader.h',
- 'cpp/url_request_info.h',
- 'cpp/url_response_info.h',
- 'cpp/var.h',
- # End ppapi_cpp.gypi
- # Updated automatically by update-scons.py.
- # From ppapi_cpp.gypi:ppapi_cpp:cpp/[^/]*\.h
- 'cpp/module_embedder.h',
- # End ppapi_cpp.gypi
- ]
+# From ppapi_cpp.gypi:ppapi_c:c/dev/[^/]*\.h
+c_dev_headers = [s for s in ppapi_c['sources']
+ if re.match('^c/dev/[^/]*\.h$', s)]
-cpp_dev_headers = [
- # Updated automatically by update-scons.py.
- # From ppapi_cpp.gypi:ppapi_cpp_objects:cpp/dev/[^/]*\.h
- 'cpp/dev/buffer_dev.h',
- 'cpp/dev/context_3d_dev.h',
- 'cpp/dev/directory_entry_dev.h',
- 'cpp/dev/directory_reader_dev.h',
- 'cpp/dev/file_chooser_dev.h',
- 'cpp/dev/file_io_dev.h',
- 'cpp/dev/file_ref_dev.h',
- 'cpp/dev/file_system_dev.h',
- 'cpp/dev/find_dev.h',
- 'cpp/dev/font_dev.h',
- 'cpp/dev/fullscreen_dev.h',
- 'cpp/dev/graphics_3d_client_dev.h',
- 'cpp/dev/graphics_3d_dev.h',
- 'cpp/dev/printing_dev.h',
- 'cpp/dev/scrollbar_dev.h',
- 'cpp/dev/selection_dev.h',
- 'cpp/dev/surface_3d_dev.h',
- 'cpp/dev/transport_dev.h',
- 'cpp/dev/url_util_dev.h',
- 'cpp/dev/video_decoder_dev.h',
- 'cpp/dev/widget_client_dev.h',
- 'cpp/dev/widget_dev.h',
- 'cpp/dev/zoom_dev.h',
- 'cpp/dev/scriptable_object_deprecated.h',
- # End ppapi_cpp.gypi
- ]
+# From ppapi_cpp.gypi:ppapi_cpp_objects:cpp/[^/]*\.h
+# From ppapi_cpp.gypi:ppapi_cpp:cpp/[^/]*\.h
+cpp_headers = ([s for s in ppapi_cpp_objects['sources']
+ if re.match('^cpp/[^/]*\.h$', s)] +
Mark Seaborn 2011/03/22 01:17:27 The '^' should be redundant in re.match(), FWIW.
bradn 2011/03/22 01:51:43 Done.
+ [s for s in ppapi_cpp['sources']
+ if re.match('^cpp/[^/]*\.h$', s)])
-gles2_headers = [
- # Updated automatically by update-scons.py.
- # From ppapi_gl.gypi:ppapi_gles2:.*\.h
- 'lib/gl/gles2/gl2ext_ppapi.h',
- # End ppapi_gl.gypi
-]
+# From ppapi_cpp.gypi:ppapi_cpp_objects:cpp/dev/[^/]*\.h
+cpp_dev_headers = [s for s in ppapi_cpp_objects['sources']
+ if re.match('^dev/[^/]*\.h$', s)]
+# From ppapi_gl.gypi:ppapi_gles2:.*\.h
+gles2_headers = [s for s in ppapi_gles2['sources']
+ if s.endswith('.h')]
+
+
c_header_install = env.AddHeaderToSdk(
['$SOURCE_ROOT/ppapi/' + h for h in c_headers], os.path.join('ppapi', 'c'))
c_dev_header_install = env.AddHeaderToSdk(

Powered by Google App Engine
This is Rietveld 408576698