Update MB for the new way to run 'analyze'.

Update MB for the new way to run 'analyze'.

Calling `mb analyze` will now get two lists of targets passed in
instead of one, and return two differently named lists as output.
See the updated docs or the bug for details.

The code currently supports both new and old interfaces, but we'll
remove the old interface once the recipes have been updated.

R=sky@chromium.org, phajdan.jr@chromium.org
BUG=552146
Committed: https://crrev.com/5ab84a5078142353428eb669e3ecf7ade00da57e
Cr-Commit-Position: refs/heads/master@{#359575}

[Dirk Pranke, 2015-11-13 02:13:45]

Patchset #1 (id:1) has been deleted

[Dirk Pranke, 2015-11-13 02:13:58]

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

[Dirk Pranke, 2015-11-13 02:14:20]

Description was changed from

==========
Update MB for the new way to run 'analyze'.

Calling `mb analyze` will now get two lists of targets passed in
instead of one, and return two differently named lists as output.
See the updated docs or the bug for details.

The code currently supports both new and old interfaces, but we'll
remove the old interface once the recipes have been updated.

R=sky@chromium.org
BUG=552146
==========

to

==========
Update MB for the new way to run 'analyze'.

Calling `mb analyze` will now get two lists of targets passed in
instead of one, and return two differently named lists as output.
See the updated docs or the bug for details.

The code currently supports both new and old interfaces, but we'll
remove the old interface once the recipes have been updated.

R=sky@chromium.org, phajdan.jr@chromium.org
BUG=552146
==========

[Dirk Pranke, 2015-11-13 02:14:20]

dpranke@chromium.org changed reviewers:
+ phajdan.jr@chromium.org

[commit-bot: I haz the power, 2015-11-13 02:14:24]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1440173002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1440173002/20001

[Dirk Pranke, 2015-11-13 02:14:28]

https://codereview.chromium.org/1440173002/diff/20001/tools/mb/docs/design_sp...
File tools/mb/docs/design_spec.md (right):

https://codereview.chromium.org/1440173002/diff/20001/tools/mb/docs/design_sp...
tools/mb/docs/design_spec.md:194: in that case.
sky@, you might want to double-check that GYP handles this properly as well.

[commit-bot: I haz the power, 2015-11-13 02:19:47]

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

[commit-bot: I haz the power, 2015-11-13 02:19:48]

Dry run: Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)

[Dirk Pranke, 2015-11-13 02:51:29]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2015-11-13 02:52:16]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1440173002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1440173002/20001

[commit-bot: I haz the power, 2015-11-13 02:52:17]

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

[commit-bot: I haz the power, 2015-11-13 02:52:17]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Dirk Pranke, 2015-11-13 03:24:50]

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

[commit-bot: I haz the power, 2015-11-13 03:25:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1440173002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1440173002/20001

[commit-bot: I haz the power, 2015-11-13 04:04:08]

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

[commit-bot: I haz the power, 2015-11-13 04:04:08]

Dry run: This issue passed the CQ dry run.

[Paweł Hajdan Jr., 2015-11-13 12:34:07]

LGTM

[sky, 2015-11-13 16:26:49]

https://codereview.chromium.org/1440173002/diff/20001/tools/mb/docs/design_sp...
File tools/mb/docs/design_spec.md (right):

https://codereview.chromium.org/1440173002/diff/20001/tools/mb/docs/design_sp...
tools/mb/docs/design_spec.md:194: in that case.
On 2015/11/13 02:14:28, Dirk Pranke wrote:
> sky@, you might want to double-check that GYP handles this properly as well.

GYP ends up generating an error in this case. I don't know how the recipe side
handles that error.

[Dirk Pranke, 2015-11-13 17:28:13]

The CQ bit was checked by dpranke@chromium.org

[commit-bot: I haz the power, 2015-11-13 17:29:25]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1440173002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1440173002/20001

[commit-bot: I haz the power, 2015-11-13 17:35:09]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2015-11-13 17:37:08]

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