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

Unified Diff: sync/sync.gyp

Issue 10970068: Do not build sync command line tools on iOS. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 3 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sync/sync.gyp
diff --git a/sync/sync.gyp b/sync/sync.gyp
index cdbbd16c0ca2723d0ebbdb928af27975cfe95091..96ccfcc363d25698e6c0b534df69ddda0b82c605 100644
--- a/sync/sync.gyp
+++ b/sync/sync.gyp
@@ -826,46 +826,49 @@
}],
],
},
-
- # A tool to listen to sync notifications and print them out.
- {
- 'target_name': 'sync_listen_notifications',
- 'type': 'executable',
- 'dependencies': [
- '../base/base.gyp:base',
- '../jingle/jingle.gyp:notifier',
- '../net/net.gyp:net',
- '../net/net.gyp:net_test_support',
- 'sync',
- 'sync_notifier',
- ],
- 'sources': [
- 'tools/sync_listen_notifications.cc',
- ],
- },
-
- # A standalone command-line sync client.
- {
- 'target_name': 'sync_client',
- 'type': 'executable',
- 'defines': [
- 'SYNC_TEST',
- ],
- 'dependencies': [
- '../base/base.gyp:base',
- '../jingle/jingle.gyp:notifier',
- '../net/net.gyp:net',
- '../net/net.gyp:net_test_support',
- 'sync',
- 'sync_notifier',
- 'syncapi_core',
- ],
- 'sources': [
- 'tools/sync_client.cc',
- ],
- },
],
'conditions': [
+ ['OS != "ios"', {
+ 'targets': [
+ # A tool to listen to sync notifications and print them out.
+ {
+ 'target_name': 'sync_listen_notifications',
+ 'type': 'executable',
+ 'dependencies': [
+ '../base/base.gyp:base',
+ '../jingle/jingle.gyp:notifier',
+ '../net/net.gyp:net',
+ '../net/net.gyp:net_test_support',
+ 'sync',
+ 'sync_notifier',
+ ],
+ 'sources': [
+ 'tools/sync_listen_notifications.cc',
+ ],
+ },
+
+ # A standalone command-line sync client.
+ {
+ 'target_name': 'sync_client',
+ 'type': 'executable',
+ 'defines': [
+ 'SYNC_TEST',
+ ],
+ 'dependencies': [
+ '../base/base.gyp:base',
+ '../jingle/jingle.gyp:notifier',
+ '../net/net.gyp:net',
+ '../net/net.gyp:net_test_support',
+ 'sync',
+ 'sync_notifier',
+ 'syncapi_core',
+ ],
+ 'sources': [
+ 'tools/sync_client.cc',
+ ],
+ },
+ ],
+ }],
# Special target to wrap a gtest_target_type==shared_library
# sync_unit_tests into an android apk for execution.
['OS == "android" and gtest_target_type == "shared_library"', {
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698