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

Unified Diff: build/common.gypi

Issue 14577007: Gyp changes for ninja iOS build. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: set generator inside mac_build 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
« no previous file with comments | « no previous file | build/ios/mac_build.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 8f479d71a3977abef592bd781e8232eb0b972075..c4123acd530922624eb268a1a834ab5a5bb075ea 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -3982,17 +3982,22 @@
},
},
},
- 'xcode_settings': {
- 'conditions': [
- ['chromium_ios_signing', {
- # iOS SDK wants everything for device signed.
- 'CODE_SIGN_IDENTITY[sdk=iphoneos*]': 'iPhone Developer',
- }, {
- 'CODE_SIGNING_REQUIRED': 'NO',
- 'CODE_SIGN_IDENTITY[sdk=iphoneos*]': '',
- }],
- ],
- },
+ 'conditions': [
+ # TODO(justincohen): ninja builds don't support signing yet.
+ ['"<(GENERATOR)"!="ninja"', {
+ 'xcode_settings': {
+ 'conditions': [
+ ['chromium_ios_signing', {
+ # iOS SDK wants everything for device signed.
+ 'CODE_SIGN_IDENTITY[sdk=iphoneos*]': 'iPhone Developer',
+ }, {
+ 'CODE_SIGNING_REQUIRED': 'NO',
+ 'CODE_SIGN_IDENTITY[sdk=iphoneos*]': '',
+ }],
+ ],
+ },
+ }],
+ ],
}],
], # target_conditions
}, # target_defaults
@@ -4341,7 +4346,14 @@
['OS=="ios"', {
'conditions': [
['ios_sdk_path==""', {
- 'SDKROOT': 'iphoneos<(ios_sdk)', # -isysroot
+ 'conditions': [
+ # TODO(justincohen): Ninja only supports simulator for now.
+ ['"<(GENERATOR)"=="ninja"', {
+ 'SDKROOT': 'iphonesimulator<(ios_sdk)', # -isysroot
+ }, {
+ 'SDKROOT': 'iphoneos<(ios_sdk)', # -isysroot
+ }],
+ ],
}, {
'SDKROOT': '<(ios_sdk_path)', # -isysroot
}],
« no previous file with comments | « no previous file | build/ios/mac_build.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698