depot_tools: add infra_paths recipe module for infra-specific paths (attempt #2)

depot_tools: add infra_paths recipe module for infra-specific paths (attempt #2)

This is different from previous attempt https://codereview.chromium.org/1915463002 .

The change is completely backwards-compatible, and doesn't introduce recipe
expectation changes.

Modules can be switched to depend on infra_paths independently,
without any expectation changes either.

By moving path module's config.py to paths_config.py here, we also guarantee
that the returned results are going to be exactly the same.

Depends on https://codereview.chromium.org/1923363003 .

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

[Paweł Hajdan Jr., 2016-04-28 13:16:19]

phajdan.jr@chromium.org changed reviewers:
+ iannucci@chromium.org, martiniss@chromium.org

[martiniss, 2016-04-28 23:08:32]

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

[commit-bot: I haz the power, 2016-04-28 23:08:35]

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

[commit-bot: I haz the power, 2016-04-28 23:15:07]

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

[commit-bot: I haz the power, 2016-04-28 23:15:08]

Dry run: Try jobs failed on following builders:
  Depot Tools Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Depot%20Tools%20Presubm...)

[martiniss, 2016-04-28 23:15:23]

lgtm

https://codereview.chromium.org/1926033002/diff/1/recipe_modules/infra_paths/...
File recipe_modules/infra_paths/example.py (right):

https://codereview.chromium.org/1926033002/diff/1/recipe_modules/infra_paths/...
recipe_modules/infra_paths/example.py:13: return
maybe add an actual example here? Like echo the slave build path or something.

[Paweł Hajdan Jr., 2016-04-29 09:34:48]

https://codereview.chromium.org/1926033002/diff/1/recipe_modules/infra_paths/...
File recipe_modules/infra_paths/example.py (right):

https://codereview.chromium.org/1926033002/diff/1/recipe_modules/infra_paths/...
recipe_modules/infra_paths/example.py:13: return
On 2016/04/28 at 23:15:23, martiniss wrote:
> maybe add an actual example here? Like echo the slave build path or something.

Done. FWIW, not sure how useful it is: repr() just repeats the name of the path.

[Paweł Hajdan Jr., 2016-04-29 09:34:54]

The CQ bit was checked by phajdan.jr@chromium.org

[Paweł Hajdan Jr., 2016-04-29 09:34:54]

The patchset sent to the CQ was uploaded after l-g-t-m from
martiniss@chromium.org
Link to the patchset: https://codereview.chromium.org/1926033002/#ps20001
(title: "fixes")

[commit-bot: I haz the power, 2016-04-29 09:35:09]

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

[commit-bot: I haz the power, 2016-04-29 09:37:43]

Description was changed from

==========
depot_tools: add infra_paths recipe module for infra-specific paths (attempt #2)

This is different from previous attempt
https://codereview.chromium.org/1915463002 .

The change is completely backwards-compatible, and doesn't introduce recipe
expectation changes.

Modules can be switched to depend on infra_paths independently,
without any expectation changes either.

By moving path module's config.py to paths_config.py here, we also guarantee
that the returned results are going to be exactly the same.

Depends on https://codereview.chromium.org/1923363003 .

BUG=chromium:605919
==========

to

==========
depot_tools: add infra_paths recipe module for infra-specific paths (attempt #2)

This is different from previous attempt
https://codereview.chromium.org/1915463002 .

The change is completely backwards-compatible, and doesn't introduce recipe
expectation changes.

Modules can be switched to depend on infra_paths independently,
without any expectation changes either.

By moving path module's config.py to paths_config.py here, we also guarantee
that the returned results are going to be exactly the same.

Depends on https://codereview.chromium.org/1923363003 .

BUG=chromium:605919

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

[commit-bot: I haz the power, 2016-04-29 09:37:44]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=300324