[Sync] Clean up sync.gyp

sync/sync.gyp

Index: sync/sync.gyp
diff --git a/sync/sync.gyp b/sync/sync.gyp
index f355761348d9ba07a05b34d67b25a0fc2ba4c1f6..fce0d2f6d9f166e8ed584012f2b93ca4595b5192 100644
--- a/sync/sync.gyp
+++ b/sync/sync.gyp
@@ -28,11 +28,8 @@
         'protocol/sync_proto.gyp:sync_proto',
       ],
       'export_dependent_settings': [
-        '../base/base.gyp:base',
-        '../build/temp_gyp/googleurl.gyp:googleurl',
-        '../crypto/crypto.gyp:crypto',
-        '../net/net.gyp:net',
-        '../sql/sql.gyp:sql',
+        # Propagate sync_proto since our headers include its generated
+        # files.
         'protocol/sync_proto.gyp:sync_proto',
       ],
       'sources': [
@@ -193,12 +190,13 @@
         '../base/base.gyp:base',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
+        'protocol/sync_proto.gyp:sync_proto',

[Ryan Sleevi, 2012/03/16 19:50:23]

Unless your header files (for this target) are directly included the protobufs,
then don't do this - Just EDS 'sync'.

[akalin, 2012/03/16 20:02:00]

yeah, they are.

         'sync',
       ],
       'export_dependent_settings': [
-        '../base/base.gyp:base',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
+        'protocol/sync_proto.gyp:sync_proto',
         'sync',
       ],
       'sources': [
@@ -250,17 +248,19 @@
       'suppress_wildcard': 1,
       'dependencies': [
         '../base/base.gyp:base',
-        '../base/base.gyp:test_support_base',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
+        'protocol/sync_proto.gyp:sync_proto',
         'sync',
         'test_support_sync',
       ],
+      # Propagate all dependencies since the actual compilation
+      # happens in the dependents.
       'export_dependent_settings': [
         '../base/base.gyp:base',
-        '../base/base.gyp:test_support_base',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
+        'protocol/sync_proto.gyp:sync_proto',

[Ryan Sleevi, 2012/03/16 19:50:23]

Same here

[akalin, 2012/03/16 20:02:00]

yeap, here too.

         'sync',
         'test_support_sync',
       ],
@@ -331,13 +331,16 @@
     {
       'target_name': 'sync_unit_tests_canary',
       'type': 'executable',
-      'sources': [
-        '../base/test/run_all_unittests.cc',
+      'include_dirs': [
+        '..',
       ],
       'dependencies': [
+        '../base/base.gyp:test_support_base',
         'sync_tests',
       ],
-
+      'sources': [
+        '../base/test/run_all_unittests.cc',
+      ],
       # TODO(akalin): This is needed because histogram.cc uses
       # leak_annotations.h, which pulls this in.  Make 'base'
       # propagate this dependency.