Don't try to delete directories if sub directories are still part of the client

Don't try to delete directories if sub directories are still part of the client

BUG=171212
TEST=ios debug simulator doesn't fail every other run
R=maruel@chromium.org

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

[M-A Ruel, 2013-03-08 15:39:40]

https://codereview.chromium.org/12537007/diff/1/gclient.py
File gclient.py (right):

https://codereview.chromium.org/12537007/diff/1/gclient.py#newcode1134
gclient.py:1134: for path in path_list:
return any(
    path[:len(parent_plus_slash)] == parent_plus_slash
    for path in path_list)

[jochen (gone - plz use gerrit), 2013-03-08 15:51:32]

https://codereview.chromium.org/12537007/diff/1/gclient.py
File gclient.py (right):

https://codereview.chromium.org/12537007/diff/1/gclient.py#newcode1134
gclient.py:1134: for path in path_list:
On 2013/03/08 15:39:41, Marc-Antoine Ruel wrote:
> return any(
>     path[:len(parent_plus_slash)] == parent_plus_slash
>     for path in path_list)

Done.

[M-A Ruel, 2013-03-08 15:53:56]

https://codereview.chromium.org/12537007/diff/3/gclient.py
File gclient.py (right):

https://codereview.chromium.org/12537007/diff/3/gclient.py#newcode1135
gclient.py:1135: path[:len(parent_plus_slash)] == parent_plus_slash
as a matter of fact;

any(path.startswith(parent + '/') for path in path_list)

so probably just put it inline with the condition, as you prefer.

[jochen (gone - plz use gerrit), 2013-03-11 07:32:10]

https://codereview.chromium.org/12537007/diff/3/gclient.py
File gclient.py (right):

https://codereview.chromium.org/12537007/diff/3/gclient.py#newcode1135
gclient.py:1135: path[:len(parent_plus_slash)] == parent_plus_slash
On 2013/03/08 15:53:56, Marc-Antoine Ruel wrote:
> as a matter of fact;
> 
> any(path.startswith(parent + '/') for path in path_list)
> 
> so probably just put it inline with the condition, as you prefer.

Done.

[M-A Ruel, 2013-03-11 13:04:23]

lgtm

[commit-bot: I haz the power, 2013-03-11 13:04:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jochen@chromium.org/12537007/9001

[commit-bot: I haz the power, 2013-03-11 13:07:44]

Change committed as 187284