Flip all but ios-simulator builders to GN.

Flip all but ios-simulator builders to GN.

This affects:

- tryserver.chromium.mac:
  - ios-device
  - ios-simulator-cronet
- chromium.mac
  - ios-device
- chromium.fyi
  - ClangToTiOS
  - EarlGreyiOS

This CL also flips the ios-device-gn builder to use the xcode version
of Clang so that we can get some coverage of that configuration.

The ios-simulator builder will remain on GYP until we drop GYP support.

R=sdefresne@chromium.org, smut@chromium.org
BUG=621708
Committed: https://crrev.com/628f62988baee46f58385de962f98409d77c17ef
Cr-Commit-Position: refs/heads/master@{#408585}

[smut, 2016-07-28 23:55:04]

smut@google.com changed reviewers:
+ smut@google.com

[smut, 2016-07-28 23:55:04]

lgtm

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.fyi...
File ios/build/bots/chromium.fyi/ClangToTiOS.json (right):

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.fyi...
ios/build/bots/chromium.fyi/ClangToTiOS.json:21: "target_cpu=\"arm\""
Sort alpha.

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
File ios/build/bots/chromium.mac/ios-device-gn.json (right):

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
ios/build/bots/chromium.mac/ios-device-gn.json:27: "use_xcode_clang=true"
Sort alpha.

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
File ios/build/bots/chromium.mac/ios-device.json (right):

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
ios/build/bots/chromium.mac/ios-device.json:17: "target_subarch=arm32"
Can we make this use target_subarch=both if it's supported now?

[Dirk Pranke, 2016-07-29 00:08:31]

thanks for the review!

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.fyi...
File ios/build/bots/chromium.fyi/ClangToTiOS.json (right):

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.fyi...
ios/build/bots/chromium.fyi/ClangToTiOS.json:21: "target_cpu=\"arm\""
On 2016/07/28 23:55:04, smut wrote:
> Sort alpha.

Will do.

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
File ios/build/bots/chromium.mac/ios-device-gn.json (right):

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
ios/build/bots/chromium.mac/ios-device-gn.json:27: "use_xcode_clang=true"
On 2016/07/28 23:55:04, smut wrote:
> Sort alpha.

Will fix.

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
File ios/build/bots/chromium.mac/ios-device.json (right):

https://codereview.chromium.org/2194703002/diff/1/ios/build/bots/chromium.mac...
ios/build/bots/chromium.mac/ios-device.json:17: "target_subarch=arm32"
On 2016/07/28 23:55:04, smut wrote:
> Can we make this use target_subarch=both if it's supported now?

Sure (assuming we can handle the load, since this is still in the CQ, but let's
give it a try).

[Dirk Pranke, 2016-07-29 00:23:24]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-07-29 00:23:25]

The patchset sent to the CQ was uploaded after l-g-t-m from smut@google.com
Link to the patchset: https://codereview.chromium.org/2194703002/#ps40001
(title: "update comment")

[commit-bot: I haz the power, 2016-07-29 00:23:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-29 00:30:10]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-29 00:30:11]

Try jobs failed on following builders:
  ios-device-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)

[Dirk Pranke, 2016-07-29 00:40:38]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-07-29 00:40:39]

The patchset sent to the CQ was uploaded after l-g-t-m from smut@google.com
Link to the patchset: https://codereview.chromium.org/2194703002/#ps60001
(title: "fix ios-device-gn to not use goma w/ xcode_clang")

[commit-bot: I haz the power, 2016-07-29 00:41:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-29 00:45:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-07-29 00:45:57]

Try jobs failed on following builders:
  ios-device-gn on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-gn/bui...)

[Dirk Pranke, 2016-07-29 04:38:23]

The CQ bit was checked by dpranke@chromium.org

[Dirk Pranke, 2016-07-29 04:38:24]

The patchset sent to the CQ was uploaded after l-g-t-m from smut@google.com
Link to the patchset: https://codereview.chromium.org/2194703002/#ps80001
(title: "fix ios-device, turn off xcode-clang for the moment")

[commit-bot: I haz the power, 2016-07-29 04:38:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-29 05:48:06]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-07-29 05:49:31]

Description was changed from

==========
Flip all but ios-simulator builders to GN.

This affects:

- tryserver.chromium.mac:
  - ios-device
  - ios-simulator-cronet
- chromium.mac
  - ios-device
- chromium.fyi
  - ClangToTiOS
  - EarlGreyiOS

This CL also flips the ios-device-gn builder to use the xcode version
of Clang so that we can get some coverage of that configuration.

The ios-simulator builder will remain on GYP until we drop GYP support.

R=sdefresne@chromium.org, smut@chromium.org
BUG=621708
==========

to

==========
Flip all but ios-simulator builders to GN.

This affects:

- tryserver.chromium.mac:
  - ios-device
  - ios-simulator-cronet
- chromium.mac
  - ios-device
- chromium.fyi
  - ClangToTiOS
  - EarlGreyiOS

This CL also flips the ios-device-gn builder to use the xcode version
of Clang so that we can get some coverage of that configuration.

The ios-simulator builder will remain on GYP until we drop GYP support.

R=sdefresne@chromium.org, smut@chromium.org
BUG=621708

Committed: https://crrev.com/628f62988baee46f58385de962f98409d77c17ef
Cr-Commit-Position: refs/heads/master@{#408585}
==========

[commit-bot: I haz the power, 2016-07-29 05:49:32]

Patchset 5 (id:??) landed as
https://crrev.com/628f62988baee46f58385de962f98409d77c17ef
Cr-Commit-Position: refs/heads/master@{#408585}

[krasin1, 2016-08-01 17:58:37]

Hi Dirk,

looks like this change has broken ios ToT bot:
https://build.chromium.org/p/chromium.fyi/builders/ClangToTiOS/builds/7908/st...

[Dirk Pranke, 2016-08-01 18:00:39]

On 2016/08/01 17:58:37, krasin1 wrote:
> Hi Dirk,
> 
> looks like this change has broken ios ToT bot:
>
https://build.chromium.org/p/chromium.fyi/builders/ClangToTiOS/builds/7908/st...

Yes, thakis has posted a fix: https://codereview.chromium.org/2199833002/