Add --reset-to-deps and improve comments

Make gclient.git.revert reset to DEPS revision

BUG=http://crosbug.com/2897

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

[djmm, 2010-04-29 22:43:45]

LGTM!  

Thanks a lot Nasser for doing this.

[Marc-Antoine Ruel (Google), 2010-04-29 23:00:29]

I don't understand, how is it different than gclient revert?

Le 29 avr. 2010 18:43, <djmm@chromium.org> a écrit :
> LGTM!
>
> Thanks a lot Nasser for doing this.
>
> http://codereview.chromium.org/1742017/show

[Nasser Grainawi, 2010-04-29 23:07:20]

Hmm, guess it shouldn't be different, though git is doing something 
different for gclient revert right now.

Djmm, will it work for you if I fix gclient revert to revert to the deps 
version for git? It already does that for svn.

Nasser

Marc-Antoine Ruel wrote:
> I don't understand, how is it different than gclient revert?
> 
> Le 29 avr. 2010 18:43, <djmm@chromium.org <mailto:djmm@chromium.org>> a 
> écrit :
>  > LGTM!
>  >
>  > Thanks a lot Nasser for doing this.
>  >
>  > http://codereview.chromium.org/1742017/show
> 

[djmm, 2010-04-29 23:17:38]

I suppose it's no different to run two commands, though as slow as gclient runs
through 50 repos, I can see the benefit of making a single pass with a cmd-line
option.

On 2010/04/29 23:07:20, Nasser Grainawi wrote:
> Hmm, guess it shouldn't be different, though git is doing something 
> different for gclient revert right now.
> 
> Djmm, will it work for you if I fix gclient revert to revert to the deps 
> version for git? It already does that for svn.
> 
> Nasser
> 
> Marc-Antoine Ruel wrote:
> > I don't understand, how is it different than gclient revert?
> > 
> > Le 29 avr. 2010 18:43, <mailto:djmm@chromium.org <mailto:djmm@chromium.org>>
a 
> > écrit :
> >  > LGTM!
> >  >
> >  > Thanks a lot Nasser for doing this.
> >  >
> >  > http://codereview.chromium.org/1742017/show
> > 
> 
>

[Nasser Grainawi, 2010-04-30 15:33:25]

M-A, PTAL

[M-A Ruel, 2010-04-30 15:46:35]

lgtm, much better than the original solution.

http://codereview.chromium.org/1742017/diff/9001/10001
File gclient_scm.py (right):

http://codereview.chromium.org/1742017/diff/9001/10001#newcode409
gclient_scm.py:409: if deps_revision.startswith('refs/heads/'):
nit: I think I'd prefer a s/^refs\/heads\//origin\// regexp but I don't really
care.

[djmm, 2010-04-30 20:02:49]

LGTM

[Nasser Grainawi, 2010-04-30 20:07:15]

djmm@chromium.org wrote:
> LGTM
> 
> http://codereview.chromium.org/1742017/show

It's failing a few tests right now that I'm trying to get fixed. Sadly 
the tests don't seem to correspond with a real gclient w/ DEPS setup...

[M-A Ruel, 2010-04-30 20:57:00]

lgtm