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

Unified Diff: base/base.gyp

Issue 10700158: Get base_unittests closer to compiling for iOS (Closed) Base URL: http://git.chromium.org/chromium/src.git@all-ios-tweak
Patch Set: Comment fix Created 8 years, 5 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 | base/test/test_suite.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/base.gyp
diff --git a/base/base.gyp b/base/base.gyp
index 1a56fb2357cc462c58eceac058b2dbbae1159a9c..40f3ce563d0644d667435cabade437efab2731d1 100644
--- a/base/base.gyp
+++ b/base/base.gyp
@@ -552,6 +552,23 @@
}],
],
}],
+ ['OS == "ios"', {
+ 'sources/': [
+ # Only test the iOS-meaningful portion of process_utils.
+ ['exclude', '^process_util_unittest'],
+ ['include', '^process_util_unittest_ios.cc$'],
+ # Requires spawning processes.
+ ['exclude', '^metrics/stats_table_unittest.cc'],
+ # TODO(ios): Remove these as base/ is unforked.
+ # For now, exclude everything that doesn't build as-is, just to
+ # get a minimal target building.
+ ['exclude', '^memory/aligned_memory_unittest.cc'],
+ ['exclude', '^message_'],
+ ['exclude', '^shared_memory_unittest.cc'],
+ ['exclude', '^sys_info_unittest.cc'],
+ ['exclude', '^system_monitor'],
+ ],
+ }],
['use_glib==1', {
'sources!': [
'file_version_info_unittest.cc',
@@ -612,7 +629,17 @@
'closure_blocks_leopard_compat',
],
}],
- ],
+ ], # conditions
+ 'target_conditions': [
rohitrao (ping after 24h) 2012/07/11 12:05:10 Why does this need to be in target_conditions inst
stuartmorgan 2012/07/11 12:56:56 Because that's the step that filename_rules is imp
+ ['OS == "ios"', {
+ 'sources/': [
+ # Pull in specific Mac files for iOS (which have been filtered out
+ # by file name rules).
+ ['include', 'mac/objc_property_releaser_unittest.mm'],
+ ['include', 'sys_string_conversions_mac_unittest.mm'],
+ ],
+ }],
+ ], # target_conditions
},
{
'target_name': 'check_example',
@@ -824,7 +851,7 @@
},
],
}],
- ['os_posix==1 and OS!="mac"', {
+ ['os_posix==1 and OS!="mac" and OS!="ios"', {
'targets': [
{
'target_name': 'symbolize',
« no previous file with comments | « no previous file | base/test/test_suite.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698