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

Unified Diff: ios/build/bots/chromium.mac/iOS_Device_GN.json

Issue 1411183010: Make MB aware of iOS bot configs and get iOS working. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rework to read GYP_DEFINES/gn_args from bot configs in MB Created 5 years, 1 month 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: ios/build/bots/chromium.mac/iOS_Device_GN.json
diff --git a/ios/build/bots/chromium.mac/iOS_Device_(ninja).json b/ios/build/bots/chromium.mac/iOS_Device_GN.json
similarity index 59%
copy from ios/build/bots/chromium.mac/iOS_Device_(ninja).json
copy to ios/build/bots/chromium.mac/iOS_Device_GN.json
index 54408813aab494e758ccd026b133cb8d3d0f71f7..6af2ec58c15de1d5a3e38f34e4060633e28bb04d 100644
--- a/ios/build/bots/chromium.mac/iOS_Device_(ninja).json
+++ b/ios/build/bots/chromium.mac/iOS_Device_GN.json
@@ -1,15 +1,22 @@
{
"owners": [
+ "dpranke",
"smut"
],
"comments": [
- "Builder for fat binaries using ninja."
+ "GN + Ninja fat binary builder."
],
"xcode version": "7.0",
"GYP_DEFINES": {
"chromium_ios_signing": "0",
"target_subarch": "both"
},
+ "gn_args": [
+ "target_os=\"ios\"",
+ "target_cpu=\"arm\"",
+ "ios_enable_code_signing=false"
smut 2015/11/18 22:00:29 Reverse the list so this is alphabetical. The esc
Dirk Pranke 2015/11/18 22:30:27 Okay. I was wondering if you'd want that or not.
+ ],
+ "mb_type": "gn",
"compiler": "ninja",
"configuration": "Release",
"sdk": "iphoneos9.0",

Powered by Google App Engine
This is Rietveld 408576698