Switch DEPS to git.

Switch DEPS to git.

BUG=415464
LOG=n
R=agable@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=25274

[Michael Achenbach, 2014-11-05 15:26:28]

machenbach@chromium.org changed reviewers:
+ agable@chromium.org, tandrii@chromium.org

[Michael Achenbach, 2014-11-05 15:26:28]

PTAL. To land on flag day.

[tandrii(chromium), 2014-11-05 15:28:39]

this patch will probably need a rebase, which will not be ff, so maybe just
re-create this CL on flag day?

[agable, 2014-11-05 19:17:40]

On 2014/11/05 at 15:28:39, tandrii wrote:
> this patch will probably need a rebase, which will not be ff, so maybe just
re-create this CL on flag day?

Did you autoformat? The formatting of the hooks list is super weird.

Also, this could totally be landed before flag day. Just because v8 is SVN
doesn't mean its deps can't be Git. We switched all of Chromium's deps well
before flag day.

Other than that, LGTM.

[Michael Achenbach, 2014-11-05 21:47:19]

I actually just copied the .DEPS.git that we had lying around. Maybe somebody
kept it up-to-date manually. Will also try rerunning deps2git and check if
there's a difference.

[agable, 2014-11-06 19:26:01]

On 2014/11/05 at 21:47:19, machenbach wrote:
> I actually just copied the .DEPS.git that we had lying around. Maybe somebody
kept it up-to-date manually. Will also try rerunning deps2git and check if
there's a difference.

Yeah, the DEPS -> .DEPS.git process does some odd formatting and leaves extra
vars lying around. This is because no one touches .DEPS.git manually, so no one
cares. For making this your real DEPS file, you should remove the excess stuff
and make sure the formatting matches the old DEPS file.

[agable, 2014-11-10 18:16:23]

On 2014/11/06 at 19:26:01, agable wrote:
> On 2014/11/05 at 21:47:19, machenbach wrote:
> > I actually just copied the .DEPS.git that we had lying around. Maybe
somebody kept it up-to-date manually. Will also try rerunning deps2git and check
if there's a difference.
> 
> Yeah, the DEPS -> .DEPS.git process does some odd formatting and leaves extra
vars lying around. This is because no one touches .DEPS.git manually, so no one
cares. For making this your real DEPS file, you should remove the excess stuff
and make sure the formatting matches the old DEPS file.

Ping; please reformat (and maybe even land) before wednesday :)

[tandrii(chromium), 2014-11-10 18:18:33]

what's the reason for landing *before* wednesday?
Is to make sure slaves pick this up in advance?

[agable, 2014-11-11 18:31:37]

On 2014/11/10 at 18:18:33, tandrii wrote:
> what's the reason for landing *before* wednesday?
> Is to make sure slaves pick this up in advance?

a) There's no reason it can't land before wednesday; it's just changing the VCS
of a few dependencies, nothing fundamental.
b) One fewer things to worry about on Wednesday. Have it done and out of the
way.

[agable, 2014-11-11 20:12:06]

https://codereview.chromium.org/702113002/diff/40001/DEPS
File DEPS (right):

https://codereview.chromium.org/702113002/diff/40001/DEPS#newcode2
DEPS:2: 'webkit_url':
This isn't used.

[Michael Achenbach, 2014-11-11 22:05:58]

All done. Kept the exact formatting as before (despite inconsistent commas,
order and white space). Can clean that up later.

[Michael Achenbach, 2014-11-11 22:07:16]

Committed patchset #5 (id:80001) manually as 25274 (presubmit successful).