git-sync-deps verifies that directory is a top-level git directory.

git-sync-deps verifies that directory is a top-level git directory.

NOTRY=true

Committed: http://code.google.com/p/skia/source/detail?r=14285

[hal.canary, 2014-04-21 18:42:52]

bad things happened to me as a result of the bug that this fixes.

[mtklein, 2014-04-21 18:44:48]

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

https://codereview.chromium.org/245503003/diff/1/tools/git-sync-deps#newcode110
tools/git-sync-deps:110: sys.stdout.write('%s\n  IS NOT TOP-LEVEL GIT
DIRECTORY.\n' % directory)
and return?

[hal.canary, 2014-04-21 18:53:33]

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

https://codereview.chromium.org/245503003/diff/1/tools/git-sync-deps#newcode110
tools/git-sync-deps:110: sys.stdout.write('%s\n  IS NOT TOP-LEVEL GIT
DIRECTORY.\n' % directory)
On 2014/04/21 18:44:48, mtklein wrote:
> and return?

Right.  That's the important bit.

[mtklein, 2014-04-21 18:55:27]

lgtm

[hal.canary, 2014-04-21 19:02:26]

The CQ bit was checked by halcanary@google.com

[commit-bot: I haz the power, 2014-04-21 19:02:39]

CQ is trying da patch. Follow status at
https://skia-tree-status.appspot.com/cq/halcanary@google.com/245503003/40001

[commit-bot: I haz the power, 2014-04-21 19:03:07]

Change committed as 14285