Have the DEPS only parsed in the relevant thread.

Have the DEPS only parsed in the relevant thread.

This means parsing the DEPS file all the time preemptively in case it's needed
for a From() reference. This could break certain checkout, we'll see.

R=dpranke@chromium.org
BUG=
TEST=


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

[Dirk Pranke, 2011-10-05 21:08:28]

lgtm

http://codereview.chromium.org/8143022/diff/1/gclient.py
File gclient.py (right):

http://codereview.chromium.org/8143022/diff/1/gclient.py#newcode345
gclient.py:345: if len(found_deps) != 1:
is it possible for us to find more than one dep with the same name?

[M-A Ruel, 2011-10-06 13:10:09]

http://codereview.chromium.org/8143022/diff/1/gclient.py
File gclient.py (right):

http://codereview.chromium.org/8143022/diff/1/gclient.py#newcode345
gclient.py:345: if len(found_deps) != 1:
On 2011/10/05 21:08:28, Dirk Pranke wrote:
> is it possible for us to find more than one dep with the same name?

In theory no. len(found_deps) should be either 0 or 1. It's just an additional
safe guard against internal borkage.

[commit-bot: I haz the power, 2011-10-10 23:11:31]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/8143022/8001

[commit-bot: I haz the power, 2011-10-10 23:17:40]

Change committed as 104817