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

Unified Diff: breakpad/breakpad.gyp

Issue 14597007: Support target/host architecture with ninja iOS builds (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 7 years, 8 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: breakpad/breakpad.gyp
diff --git a/breakpad/breakpad.gyp b/breakpad/breakpad.gyp
index 80fd6775e2fd253812c716c69bf8f8d74b57c5cc..d4c0be2519e0e7d3d6bb70e691ff038033c32a6b 100644
--- a/breakpad/breakpad.gyp
+++ b/breakpad/breakpad.gyp
@@ -105,7 +105,7 @@
},
],
}],
- [ 'OS=="mac"', {
+ [ 'OS=="mac" or (OS=="ios" and "<(GENERATOR)"=="ninja")', {
'target_defaults': {
'include_dirs': [
'src',
@@ -206,6 +206,7 @@
{
'target_name': 'dump_syms',
'type': 'executable',
+ 'toolsets': ['host','target'],
stuartmorgan 2013/05/06 11:56:35 spaces after ,
justincohen 2013/05/06 14:17:36 Done.
'include_dirs++': [
# ++ ensures this comes before src brought in from target_defaults.
'pending/src',
@@ -274,6 +275,7 @@
{
'target_name': 'symupload',
'type': 'executable',
+ 'toolsets': ['host','target'],
'include_dirs': [
'src/common/mac',
],
@@ -667,7 +669,82 @@
},
],
}],
- [ 'OS=="ios"', {
+ ['OS=="ios"', {
+ 'targets': [
+ {
+ 'target_name': 'breakpad_client',
+ 'type': 'static_library',
+ 'sources': [
+ 'src/client/ios/Breakpad.h',
+ 'src/client/ios/Breakpad.mm',
+ 'src/client/ios/BreakpadController.h',
+ 'src/client/ios/BreakpadController.mm',
+ 'src/client/ios/handler/ios_exception_minidump_generator.mm',
+ 'src/client/ios/handler/ios_exception_minidump_generator.h',
+ 'src/client/mac/crash_generation/ConfigFile.h',
+ 'src/client/mac/crash_generation/ConfigFile.mm',
+ 'src/client/mac/handler/breakpad_nlist_64.cc',
+ 'src/client/mac/handler/breakpad_nlist_64.h',
+ 'src/client/mac/handler/dynamic_images.cc',
+ 'src/client/mac/handler/dynamic_images.h',
+ 'src/client/mac/handler/protected_memory_allocator.cc',
+ 'src/client/mac/handler/protected_memory_allocator.h',
+ 'src/client/mac/handler/exception_handler.cc',
+ 'src/client/mac/handler/exception_handler.h',
+ 'src/client/mac/handler/minidump_generator.cc',
+ 'src/client/mac/handler/minidump_generator.h',
+ 'src/client/mac/sender/uploader.h',
+ 'src/client/mac/sender/uploader.mm',
+ 'src/client/minidump_file_writer.cc',
+ 'src/client/minidump_file_writer.h',
+ 'src/client/minidump_file_writer-inl.h',
+ 'src/common/convert_UTF.c',
+ 'src/common/convert_UTF.h',
+ 'src/common/mac/file_id.cc',
+ 'src/common/mac/file_id.h',
+ 'src/common/mac/HTTPMultipartUpload.m',
+ 'src/common/mac/macho_id.cc',
+ 'src/common/mac/macho_id.h',
+ 'src/common/mac/macho_utilities.cc',
+ 'src/common/mac/macho_utilities.h',
+ 'src/common/mac/macho_walker.cc',
+ 'src/common/mac/macho_walker.h',
+ 'src/common/mac/string_utilities.cc',
+ 'src/common/mac/string_utilities.h',
+ 'src/common/md5.cc',
+ 'src/common/md5.h',
+ 'src/common/simple_string_dictionary.cc',
+ 'src/common/simple_string_dictionary.h',
+ 'src/common/string_conversion.cc',
+ 'src/common/string_conversion.h',
+ 'src/google_breakpad/common/minidump_format.h',
+ ],
+ 'include_dirs': [
+ 'src',
+ 'src/client/mac/Framework',
+ 'src/common/mac',
+ # For GTMLogger.
+ '<(DEPTH)/third_party/GTM',
+ '<(DEPTH)/third_party/GTM/Foundation',
+ ],
+ 'link_settings': {
+ # Build the version of GTMLogger.m in third_party rather than the
+ # one in src/common/mac because the former catches all exceptions
+ # whereas the latter lets them propagate, which can cause odd
+ # crashes.
+ 'sources': [
+ '<(DEPTH)/third_party/GTM/Foundation/GTMLogger.h',
+ '<(DEPTH)/third_party/GTM/Foundation/GTMLogger.m',
+ ],
+ 'include_dirs': [
+ '<(DEPTH)/third_party/GTM',
+ '<(DEPTH)/third_party/GTM/Foundation',
+ ],
+ },
+ }
+ ]
+ }],
+ ['OS=="ios" and "<(GENERATOR)"!="ninja"', {
'variables': {
'ninja_output_dir': 'ninja-breakpad',
'ninja_product_dir':
@@ -756,78 +833,7 @@
'dependencies': [
'breakpad_utilities',
],
- },
- {
- 'target_name': 'breakpad_client',
- 'type': 'static_library',
- 'sources': [
- 'src/client/ios/Breakpad.h',
- 'src/client/ios/Breakpad.mm',
- 'src/client/ios/BreakpadController.h',
- 'src/client/ios/BreakpadController.mm',
- 'src/client/ios/handler/ios_exception_minidump_generator.mm',
- 'src/client/ios/handler/ios_exception_minidump_generator.h',
- 'src/client/mac/crash_generation/ConfigFile.h',
- 'src/client/mac/crash_generation/ConfigFile.mm',
- 'src/client/mac/handler/breakpad_nlist_64.cc',
- 'src/client/mac/handler/breakpad_nlist_64.h',
- 'src/client/mac/handler/dynamic_images.cc',
- 'src/client/mac/handler/dynamic_images.h',
- 'src/client/mac/handler/protected_memory_allocator.cc',
- 'src/client/mac/handler/protected_memory_allocator.h',
- 'src/client/mac/handler/exception_handler.cc',
- 'src/client/mac/handler/exception_handler.h',
- 'src/client/mac/handler/minidump_generator.cc',
- 'src/client/mac/handler/minidump_generator.h',
- 'src/client/mac/sender/uploader.h',
- 'src/client/mac/sender/uploader.mm',
- 'src/client/minidump_file_writer.cc',
- 'src/client/minidump_file_writer.h',
- 'src/client/minidump_file_writer-inl.h',
- 'src/common/convert_UTF.c',
- 'src/common/convert_UTF.h',
- 'src/common/mac/file_id.cc',
- 'src/common/mac/file_id.h',
- 'src/common/mac/HTTPMultipartUpload.m',
- 'src/common/mac/macho_id.cc',
- 'src/common/mac/macho_id.h',
- 'src/common/mac/macho_utilities.cc',
- 'src/common/mac/macho_utilities.h',
- 'src/common/mac/macho_walker.cc',
- 'src/common/mac/macho_walker.h',
- 'src/common/mac/string_utilities.cc',
- 'src/common/mac/string_utilities.h',
- 'src/common/md5.cc',
- 'src/common/md5.h',
- 'src/common/simple_string_dictionary.cc',
- 'src/common/simple_string_dictionary.h',
- 'src/common/string_conversion.cc',
- 'src/common/string_conversion.h',
- 'src/google_breakpad/common/minidump_format.h',
- ],
- 'include_dirs': [
- 'src',
- 'src/client/mac/Framework',
- 'src/common/mac',
- # For GTMLogger.
- '<(DEPTH)/third_party/GTM',
- '<(DEPTH)/third_party/GTM/Foundation',
- ],
- 'link_settings': {
- # Build the version of GTMLogger.m in third_party rather than the
- # one in src/common/mac because the former catches all exceptions
- # whereas the latter lets them propagate, which can cause odd
- # crashes.
- 'sources': [
- '<(DEPTH)/third_party/GTM/Foundation/GTMLogger.h',
- '<(DEPTH)/third_party/GTM/Foundation/GTMLogger.m',
- ],
- 'include_dirs': [
- '<(DEPTH)/third_party/GTM',
- '<(DEPTH)/third_party/GTM/Foundation',
- ],
- },
- },
+ }
],
}],
],

Powered by Google App Engine
This is Rietveld 408576698