Update recipes/recipe_modules in build/ for indexing output placeholders by names.

Update recipes/recipe_modules in build/ for indexing output placeholders by names.

The CL to index output placeholders by name is https://codereview.chromium.org/1773273003/.

BUG=chromium:593198
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299447

[stgao, 2016-03-22 05:37:48]

Patchset #1 (id:1) has been deleted

[stgao, 2016-03-22 05:39:57]

Description was changed from

==========
Update recipes/recipe_modules for named placeholder outputs.


workaround.

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

to

==========
Update recipes/recipe_modules in build/ for indexing output placeholders by
names.

The CL to index output placeholders by name is
https://codereview.chromium.org/1773273003/.

BUG=593198
==========

[stgao, 2016-03-22 05:41:02]

stgao@chromium.org changed reviewers:
+ iannucci@chromium.org, martiniss@chromium.org

[stgao, 2016-03-22 05:58:17]

PTAL

[iannucci, 2016-03-22 22:24:12]

lgtm

[stgao, 2016-03-23 05:39:31]

stgao@chromium.org changed reviewers:
+ mmeade@chromium.org

[stgao, 2016-03-23 05:39:32]

+mmeade, would you mind a review of the changes to legion? Is the change
expected? Or there was a bug?

Context: after https://codereview.chromium.org/1773273003/, stdout and stderr of
a step won't be added to the step result if no corresponding placeholders are
provided for them.

[Mike Meade, 2016-03-23 17:52:31]

lgtm

[stgao, 2016-03-23 19:33:09]

Description was changed from

==========
Update recipes/recipe_modules in build/ for indexing output placeholders by
names.

The CL to index output placeholders by name is
https://codereview.chromium.org/1773273003/.

BUG=593198
==========

to

==========
Update recipes/recipe_modules in build/ for indexing output placeholders by
names.

The CL to index output placeholders by name is
https://codereview.chromium.org/1773273003/.

BUG=chromium:593198
==========

[stgao, 2016-03-23 19:33:23]

The CQ bit was checked by stgao@chromium.org

[stgao, 2016-03-23 19:33:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org, mmeade@chromium.org
Link to the patchset: https://codereview.chromium.org/1820073003/#ps100001
(title: "Roll recipe_engine and depot_tools")

[commit-bot: I haz the power, 2016-03-23 19:33:26]

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

[commit-bot: I haz the power, 2016-03-23 19:37:43]

Description was changed from

==========
Update recipes/recipe_modules in build/ for indexing output placeholders by
names.

The CL to index output placeholders by name is
https://codereview.chromium.org/1773273003/.

BUG=chromium:593198
==========

to

==========
Update recipes/recipe_modules in build/ for indexing output placeholders by
names.

The CL to index output placeholders by name is
https://codereview.chromium.org/1773273003/.

BUG=chromium:593198

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=299447
==========

[commit-bot: I haz the power, 2016-03-23 19:37:44]

Committed patchset #5 (id:100001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=299447