Componentize protobuf_lite

third_party/protobuf/protobuf.gyp

Index: third_party/protobuf/protobuf.gyp
diff --git a/third_party/protobuf/protobuf.gyp b/third_party/protobuf/protobuf.gyp
index f9a624fb7e23959dcab6b6ea9fabeb33bcf4eed5..09f9839891a1e0d1fdffd1e4ed480a157131e0df 100644
--- a/third_party/protobuf/protobuf.gyp
+++ b/third_party/protobuf/protobuf.gyp
@@ -1,4 +1,4 @@
-# Copyright (c) 2012 The Chromium Authors. All rights reserved.
+# Copyright 2013 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.
 
@@ -129,74 +129,23 @@
         # to your .proto file.
         {
           'target_name': 'protobuf_lite',
-          'type': 'static_library',
+          'type': '<(component)',
           'toolsets': ['host', 'target'],
-          'sources': [
-            'src/google/protobuf/stubs/atomicops.h',
-            'src/google/protobuf/stubs/atomicops_internals_arm_gcc.h',
-            'src/google/protobuf/stubs/atomicops_internals_atomicword_compat.h',
-            'src/google/protobuf/stubs/atomicops_internals_macosx.h',
-            'src/google/protobuf/stubs/atomicops_internals_mips_gcc.h',
-            'src/google/protobuf/stubs/atomicops_internals_x86_gcc.cc',
-            'src/google/protobuf/stubs/atomicops_internals_x86_gcc.h',
-            'src/google/protobuf/stubs/atomicops_internals_x86_msvc.cc',
-            'src/google/protobuf/stubs/atomicops_internals_x86_msvc.h',
-            'src/google/protobuf/stubs/common.h',
-            'src/google/protobuf/stubs/once.h',
-            'src/google/protobuf/stubs/platform_macros.h',
-            'src/google/protobuf/extension_set.h',
-            'src/google/protobuf/generated_message_util.h',
-            'src/google/protobuf/message_lite.h',
-            'src/google/protobuf/repeated_field.h',
-            'src/google/protobuf/unknown_field_set.cc',
-            'src/google/protobuf/unknown_field_set.h',
-            'src/google/protobuf/wire_format_lite.h',
-            'src/google/protobuf/wire_format_lite_inl.h',
-            'src/google/protobuf/io/coded_stream.h',
-            'src/google/protobuf/io/zero_copy_stream.h',
-            'src/google/protobuf/io/zero_copy_stream_impl_lite.h',
-
-            'src/google/protobuf/stubs/common.cc',
-            'src/google/protobuf/stubs/once.cc',
-            'src/google/protobuf/stubs/hash.h',
-            'src/google/protobuf/stubs/map-util.h',
-            'src/google/protobuf/stubs/stl_util-inl.h',
-            'src/google/protobuf/extension_set.cc',
-            'src/google/protobuf/generated_message_util.cc',
-            'src/google/protobuf/message_lite.cc',
-            'src/google/protobuf/repeated_field.cc',
-            'src/google/protobuf/wire_format_lite.cc',
-            'src/google/protobuf/io/coded_stream.cc',
-            'src/google/protobuf/io/coded_stream_inl.h',
-            'src/google/protobuf/io/zero_copy_stream.cc',
-            'src/google/protobuf/io/zero_copy_stream_impl_lite.cc',
-            '<(config_h_dir)/config.h',
-          ],
-          'include_dirs': [
-            '<(config_h_dir)',
-            'src',
+          'includes': [
+            'protobuf_lite.gypi',
           ],
-          # This macro must be defined to suppress the use of dynamic_cast<>,
-          # which requires RTTI.
+          # Required for component builds. See http://crbug.com/172800.
           'defines': [
-            'GOOGLE_PROTOBUF_NO_RTTI',
-            'GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER',
+            'LIBPROTOBUF_EXPORTS',
+            'PROTOBUF_USE_DLLS',
           ],
-
           'direct_dependent_settings': {
-            'include_dirs': [
-              '<(config_h_dir)',
-              'src',
-            ],
             'defines': [
-              'GOOGLE_PROTOBUF_NO_RTTI',
-              'GOOGLE_PROTOBUF_NO_STATIC_INITIALIZER',
+              'PROTOBUF_USE_DLLS',
             ],
-            # TODO(jschuh): http://crbug.com/167187 size_t -> int.
-            'msvs_disabled_warnings': [ 4267 ],
           },
         },
-        # This is the full, heavy protobuf lib that's needed for c++ .proto's
+        # This is the full, heavy protobuf lib that's needed for c++ .protos
         # that don't specify the LITE_RUNTIME option.  The protocol
         # compiler itself (protoc) falls into that category.
         #
@@ -205,56 +154,9 @@
           'target_name': 'protobuf_full_do_not_use',
           'type': 'static_library',
           'toolsets': ['host','target'],
-          'sources': [
-            'src/google/protobuf/descriptor.h',
-            'src/google/protobuf/descriptor.pb.h',
-            'src/google/protobuf/descriptor_database.h',
-            'src/google/protobuf/dynamic_message.h',
-            'src/google/protobuf/generated_message_reflection.h',
-            'src/google/protobuf/message.h',
-            'src/google/protobuf/reflection_ops.h',
-            'src/google/protobuf/service.h',
-            'src/google/protobuf/text_format.h',
-            'src/google/protobuf/wire_format.h',
-            'src/google/protobuf/io/gzip_stream.h',
-            'src/google/protobuf/io/printer.h',
-            'src/google/protobuf/io/tokenizer.h',
-            'src/google/protobuf/io/zero_copy_stream_impl.h',
-            'src/google/protobuf/compiler/code_generator.h',
-            'src/google/protobuf/compiler/command_line_interface.h',
-            'src/google/protobuf/compiler/importer.h',
-            'src/google/protobuf/compiler/parser.h',
-
-            'src/google/protobuf/stubs/strutil.cc',
-            'src/google/protobuf/stubs/strutil.h',
-            'src/google/protobuf/stubs/substitute.cc',
-            'src/google/protobuf/stubs/substitute.h',
-            'src/google/protobuf/stubs/structurally_valid.cc',
-            'src/google/protobuf/descriptor.cc',
-            'src/google/protobuf/descriptor.pb.cc',
-            'src/google/protobuf/descriptor_database.cc',
-            'src/google/protobuf/dynamic_message.cc',
-            'src/google/protobuf/extension_set_heavy.cc',
-            'src/google/protobuf/generated_message_reflection.cc',
-            'src/google/protobuf/message.cc',
-            'src/google/protobuf/reflection_ops.cc',
-            'src/google/protobuf/service.cc',
-            'src/google/protobuf/text_format.cc',
-            'src/google/protobuf/wire_format.cc',
-            # This file pulls in zlib, but it's not actually used by protoc, so
-            # instead of compiling zlib for the host, let's just exclude this.
-            # 'src/src/google/protobuf/io/gzip_stream.cc',
-            'src/google/protobuf/io/printer.cc',
-            'src/google/protobuf/io/tokenizer.cc',
-            'src/google/protobuf/io/zero_copy_stream_impl.cc',
-            'src/google/protobuf/compiler/importer.cc',
-            'src/google/protobuf/compiler/parser.cc',
-          ],
-          'dependencies': [
-            'protobuf_lite',
-          ],
-          'export_dependent_settings': [
-            'protobuf_lite',
+          'includes': [
+            'protobuf_full_do_not_use.gypi',

[Ryan Sleevi, 2013/01/30 00:24:00]

It shouldn't be necessary to pull this into its own .gypi. I would rather keep
this in the .gyp file as part of the target.

[Raghu Simha, 2013/01/30 00:48:48]

Fair point. Restored.

+            'protobuf_lite.gypi',
           ],
         },
         {