Try again to flip the official mac builders to GN.

Try again to flip the official mac builders to GN.

R=rsesek@chromium.org
BUG=623685
Committed: https://crrev.com/b231bcf64216a6d0d52f9e1017560c0d4720a5f3
Cr-Commit-Position: refs/heads/master@{#406294}

[Dirk Pranke, 2016-07-18 23:13:24]

The CQ bit was checked by dpranke@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-18 23:14:01]

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

[Robert Sesek, 2016-07-18 23:18:58]

https://codereview.chromium.org/2155353002/diff/1/tools/mb/mb_config.pyl
File tools/mb/mb_config.pyl (right):

https://codereview.chromium.org/2155353002/diff/1/tools/mb/mb_config.pyl#newc...
tools/mb/mb_config.pyl:17: 'Mac': 'noswarming_gn_release_bot_mac_strip',
Did you want to set save_unstripped_output = true in this CL or in a follow-up
for this config?

[Dirk Pranke, 2016-07-18 23:27:57]

On 2016/07/18 23:18:58, Robert Sesek wrote:
> https://codereview.chromium.org/2155353002/diff/1/tools/mb/mb_config.pyl
> File tools/mb/mb_config.pyl (right):
> 
>
https://codereview.chromium.org/2155353002/diff/1/tools/mb/mb_config.pyl#newc...
> tools/mb/mb_config.pyl:17: 'Mac': 'noswarming_gn_release_bot_mac_strip',
> Did you want to set save_unstripped_output = true in this CL or in a follow-up
> for this config?

Let's do that in a follow up CL; it seems like we don't really need the
save_unstripped_output
arg and could just do if (enable_stripping && !enable_dsyms) ...

[Robert Sesek, 2016-07-18 23:31:47]

OK, lgtm

[Dirk Pranke, 2016-07-19 00:14:03]

On 2016/07/18 23:31:47, Robert Sesek wrote:
> OK, lgtm

Follow-up CL: https://codereview.chromium.org/2163533002/

[commit-bot: I haz the power, 2016-07-19 02:47:23]

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

[commit-bot: I haz the power, 2016-07-19 02:47:24]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2016-07-19 03:20:34]

I'm going to wait until morning to land this at this point ...

[Dirk Pranke, 2016-07-19 16:36:12]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2016-07-19 16:36:33]

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-19 16:42:20]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-19 16:42:25]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-19 16:45:14]

Description was changed from

==========
Try again to flip the official mac builders to GN.

R=rsesek@chromium.org
BUG=623685
==========

to

==========
Try again to flip the official mac builders to GN.

R=rsesek@chromium.org
BUG=623685

Committed: https://crrev.com/b231bcf64216a6d0d52f9e1017560c0d4720a5f3
Cr-Commit-Position: refs/heads/master@{#406294}
==========

[commit-bot: I haz the power, 2016-07-19 16:45:15]

Patchset 1 (id:??) landed as
https://crrev.com/b231bcf64216a6d0d52f9e1017560c0d4720a5f3
Cr-Commit-Position: refs/heads/master@{#406294}