Add gclient recurse

Add gclient recurse

This allows to recurse in all the entries to operate on them.

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

[M-A Ruel, 2010-07-01 18:30:09]

I want a small smoke test though. It's fairly easy to write now.

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

http://codereview.chromium.org/2862039/diff/1/2#newcode864
gclient.py:864: Runs a shell command on all entries.
please keep aligned.

[piman, 2010-07-01 19:36:48]

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

http://codereview.chromium.org/2862039/diff/1/2#newcode864
gclient.py:864: Runs a shell command on all entries.
On 2010/07/01 18:30:09, Marc-Antoine Ruel wrote:
> please keep aligned.

Done. The rest of the file isn't consistent though...

[M-A Ruel, 2010-07-01 19:50:26]

lgtm with small nits

http://codereview.chromium.org/2862039/diff/5001/6001
File gclient.py (right):

http://codereview.chromium.org/2862039/diff/5001/6001#newcode871
gclient.py:871: root, entries = gclient_utils.GetGClientRootAndEntries()
I'll change that to client.tree() once the other CL is in. For now it's fine.

http://codereview.chromium.org/2862039/diff/5001/6001#newcode877
gclient.py:877: env = os.environ
env = os.environ.copy() just in case

http://codereview.chromium.org/2862039/diff/5001/6002
File tests/gclient_smoketest.py (right):

http://codereview.chromium.org/2862039/diff/5001/6002#newcode675
tests/gclient_smoketest.py:675: entries = set()
I'd prefer a sorted list than a set, for the reason that running the same deps 2
times is a bug. You can just change line 693 to
self.assertEquals(sorted(entries), sorted(expected))

[piman, 2010-07-01 20:28:53]

http://codereview.chromium.org/2862039/diff/5001/6001
File gclient.py (right):

http://codereview.chromium.org/2862039/diff/5001/6001#newcode877
gclient.py:877: env = os.environ
On 2010/07/01 19:50:26, Marc-Antoine Ruel wrote:
> env = os.environ.copy() just in case

Done.

http://codereview.chromium.org/2862039/diff/5001/6002
File tests/gclient_smoketest.py (right):

http://codereview.chromium.org/2862039/diff/5001/6002#newcode675
tests/gclient_smoketest.py:675: entries = set()
On 2010/07/01 19:50:26, Marc-Antoine Ruel wrote:
> I'd prefer a sorted list than a set, for the reason that running the same deps
2
> times is a bug. You can just change line 693 to
> self.assertEquals(sorted(entries), sorted(expected))

Done.