tools/git-sync-deps: speed up common case by skipping fetch

tools/git-sync-deps: speed up common case by skipping fetch

Committed: https://skia.googlesource.com/skia/+/6aff54c32283614dc5d5acde82aef6919bc166da

[hal.canary, 2015-11-03 17:21:25]

halcanary@google.com changed reviewers:
+ mtklein@google.com

[hal.canary, 2015-11-03 17:21:25]

PTAL

[mtklein, 2015-11-03 17:29:29]

https://codereview.chromium.org/1431713002/diff/1/tools/git-sync-deps
File tools/git-sync-deps (right):

https://codereview.chromium.org/1431713002/diff/1/tools/git-sync-deps#newcode138
tools/git-sync-deps:138: subprocess.check_call(
if 0 == subprocess.call(...):
  if verbose:
    ...
  return

?

[hal.canary, 2015-11-03 17:35:15]

https://codereview.chromium.org/1431713002/diff/1/tools/git-sync-deps
File tools/git-sync-deps (right):

https://codereview.chromium.org/1431713002/diff/1/tools/git-sync-deps#newcode138
tools/git-sync-deps:138: subprocess.check_call(
On 2015/11/03 at 17:29:29, mtklein wrote:
> if 0 == subprocess.call(...):
>   if verbose:
>     ...
>   return
> 
> ?

done

[mtklein, 2015-11-03 17:38:18]

lgtm

[mtklein, 2015-11-03 17:38:22]

The CQ bit was checked by mtklein@google.com

[commit-bot: I haz the power, 2015-11-03 17:38:27]

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

[commit-bot: I haz the power, 2015-11-03 17:50:06]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/6aff54c32283614dc5d5acde82aef6919bc166da