MB: Add initial mb_config.pyl configuration file.

MB: Add initial mb_config.pyl configuration file.

This is only needed and used by the iOS GN bots so far, but
more bots will start using this in the future as we'll be migrating
from GYP to GN at some point.

Buildbot configuration is done in
https://codereview.chromium.org/1730353002

BUG=589510
R=dpranke@chromium.org

Committed: https://crrev.com/861dcb7eddd295b529120219794c246a8a1dcccf
Cr-Commit-Position: refs/heads/master@{#11752}

[kjellander_webrtc, 2016-02-24 18:08:04]

Patchset #1 (id:1) has been deleted

[kjellander_webrtc, 2016-02-24 18:11:06]

Patchset #1 (id:20001) has been deleted

[kjellander_webrtc, 2016-02-24 18:16:40]

Description was changed from

==========
MB: Add initial mb_config.pyl configuration file.

This is only needed and used by the iOS GN bots so far, but
more bots will start using this in the future as we'll be migrating
from GYP to GN at some point.

BUG=589510
==========

to

==========
MB: Add initial mb_config.pyl configuration file.

This is only needed and used by the iOS GN bots so far, but
more bots will start using this in the future as we'll be migrating
from GYP to GN at some point.

Buildbot configuration is done in
https://codereview.chromium.org/1730353002

BUG=589510
==========

[kjellander_webrtc, 2016-02-24 18:16:40]

kjellander@webrtc.org changed reviewers:
+ dpranke@chromium.org

[kjellander_webrtc, 2016-02-24 18:18:42]

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/PRESUBMIT.py
File webrtc/build/PRESUBMIT.py (right):

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/PRESUBMIT.py...
webrtc/build/PRESUBMIT.py:14: mb_script =
input_api.os_path.join(input_api.PresubmitLocalPath(), '..',
This was neat; that it was so easy to reuse the mb.py check!

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/mb_config.pyl
File webrtc/build/mb_config.pyl (right):

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/mb_config.py...
webrtc/build/mb_config.pyl:58: # TODO(kjellander): Figure out if this exists for
GN yet.
I couldn't find this setting in GN, is there one?

[kjellander_webrtc, 2016-02-24 18:33:03]

Found the ios_enable_code_signing=false setting...

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/mb_config.pyl
File webrtc/build/mb_config.pyl (right):

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/mb_config.py...
webrtc/build/mb_config.pyl:58: # TODO(kjellander): Figure out if this exists for
GN yet.
On 2016/02/24 18:18:42, kjellander (webrtc) wrote:
> I couldn't find this setting in GN, is there one?

Ah, found it: ios_enable_code_signing=false

[Dirk Pranke, 2016-02-24 20:07:29]

lgtm.

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/PRESUBMIT.py
File webrtc/build/PRESUBMIT.py (right):

https://codereview.webrtc.org/1735593002/diff/40001/webrtc/build/PRESUBMIT.py...
webrtc/build/PRESUBMIT.py:14: mb_script =
input_api.os_path.join(input_api.PresubmitLocalPath(), '..',
On 2016/02/24 18:18:42, kjellander (webrtc) wrote:
> This was neat; that it was so easy to reuse the mb.py check!

:)

[commit-bot: I haz the power, 2016-02-24 20:37:03]

Description was changed from

==========
MB: Add initial mb_config.pyl configuration file.

This is only needed and used by the iOS GN bots so far, but
more bots will start using this in the future as we'll be migrating
from GYP to GN at some point.

Buildbot configuration is done in
https://codereview.chromium.org/1730353002

BUG=589510
==========

to

==========
MB: Add initial mb_config.pyl configuration file.

This is only needed and used by the iOS GN bots so far, but
more bots will start using this in the future as we'll be migrating
from GYP to GN at some point.

Buildbot configuration is done in
https://codereview.chromium.org/1730353002

BUG=589510
R=dpranke@chromium.org

Committed: https://crrev.com/861dcb7eddd295b529120219794c246a8a1dcccf
Cr-Commit-Position: refs/heads/master@{#11752}
==========

[commit-bot: I haz the power, 2016-02-24 20:37:04]

Patchset 2 (id:??) landed as
https://crrev.com/861dcb7eddd295b529120219794c246a8a1dcccf
Cr-Commit-Position: refs/heads/master@{#11752}