[mb] Fix shared-library bots.

[mb] Fix shared-library bots.

Because the release_bot mixin includes the 'static' config,
the shared-library bots included both 'static' and 'shared'.

This removes the wrong 'static' config.

BUG=chromium:474921
TBR=vogelheim@chromium.org, jochen@chromium.org

Committed: https://crrev.com/00c51b99edf4657a0f5bd6060b9666cf4ca1e868
Cr-Commit-Position: refs/heads/master@{#37556}

[Michael Achenbach, 2016-07-06 14:26:31]

Description was changed from

==========
[mb] Fix shared-library bots.

BUG=
==========

to

==========
[mb] Fix shared-library bots.

Because the release_bot mixin includes the 'static' config,
the shared-library bots included both 'static' and 'shared'.

This removes the wrong 'static' config.

BUG=chromium:474921
TBR=vogelheim@chromium.org, jochen@chromium.org
==========

[Michael Achenbach, 2016-07-06 14:26:31]

machenbach@chromium.org changed reviewers:
+ jochen@chromium.org, vogelheim@chromium.org

[Michael Achenbach, 2016-07-06 14:26:36]

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

[commit-bot: I haz the power, 2016-07-06 14:26:38]

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

[Michael Achenbach, 2016-07-06 14:28:34]

TBR - I was afraid that bugs like that would happen. Well, now I detected this
one.

My infra-side diff tool didn't take duplicate keys into account apparently. All
gyp flags on the infra side are in a dict and the latter 'component' key
overwrote the former. But in mb, you can use the same key twice.

[Michael Achenbach, 2016-07-06 14:29:46]

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

https://codereview.chromium.org/2123243002/diff/1/infra/mb/mb_config.pyl#newc...
infra/mb/mb_config.pyl:476: 'mixins': ['release', 'static', 'goma'],
The 'static' here is the problem when using 'release_bot' together with
'shared'.

[tandrii(chromium), 2016-07-06 14:32:02]

tandrii@chromium.org changed reviewers:
+ tandrii@chromium.org

[tandrii(chromium), 2016-07-06 14:32:03]

lgtm

[Michael Achenbach, 2016-07-06 14:33:57]

The CQ bit was unchecked by machenbach@chromium.org

[Michael Achenbach, 2016-07-06 14:34:04]

The CQ bit was checked by machenbach@chromium.org

[commit-bot: I haz the power, 2016-07-06 14:34:08]

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-06 15:07:01]

Description was changed from

==========
[mb] Fix shared-library bots.

Because the release_bot mixin includes the 'static' config,
the shared-library bots included both 'static' and 'shared'.

This removes the wrong 'static' config.

BUG=chromium:474921
TBR=vogelheim@chromium.org, jochen@chromium.org
==========

to

==========
[mb] Fix shared-library bots.

Because the release_bot mixin includes the 'static' config,
the shared-library bots included both 'static' and 'shared'.

This removes the wrong 'static' config.

BUG=chromium:474921
TBR=vogelheim@chromium.org, jochen@chromium.org
==========

[commit-bot: I haz the power, 2016-07-06 15:07:02]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-06 15:07:03]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-06 15:07:43]

Description was changed from

==========
[mb] Fix shared-library bots.

Because the release_bot mixin includes the 'static' config,
the shared-library bots included both 'static' and 'shared'.

This removes the wrong 'static' config.

BUG=chromium:474921
TBR=vogelheim@chromium.org, jochen@chromium.org
==========

to

==========
[mb] Fix shared-library bots.

Because the release_bot mixin includes the 'static' config,
the shared-library bots included both 'static' and 'shared'.

This removes the wrong 'static' config.

BUG=chromium:474921
TBR=vogelheim@chromium.org, jochen@chromium.org

Committed: https://crrev.com/00c51b99edf4657a0f5bd6060b9666cf4ca1e868
Cr-Commit-Position: refs/heads/master@{#37556}
==========

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

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