Re: [PATCH] prepare deprecation of git-revert
- From
Pierre Habouzit <madcoder@debian.org>
- Date
- Oct 31, 2008, 16:58 UTC
- Message-ID
- <20081031165814.GD627@artemis.corp>
- In-Reply-To
- <20081031165003.GA5355@steel.home>
On Fri, Oct 31, 2008 at 04:50:03PM +0000, Alex Riesen wrote:
Show 11 quoted lines
> Pierre Habouzit, Fri, Oct 31, 2008 16:55:27 +0100:
> > @@ -439,16 +436,17 @@ static int revert_or_cherry_pick(int argc, const char **argv)
> >
> > int cmd_revert(int argc, const char **argv, const char *prefix)
> > {
> > +#if 0
> > + warning("git revert is deprecated, please use git cherry-pick --revert/-R instead");
> > +#endif
>
> "git revert" is much shorter to type than "git cherry-pick -R".
> How about renaming "cherry-pick" into something short, like "pick"?Do you really use git revert _that_ often ? I don't. And cherry-pick is a really usual name for the tool.
FWIW the basic idea is to deprecate revert in a (not so ?) long time, and leave git revert unimplemented for ever so that people that would like it to be 'git checkout HEAD --' alias it to that, and the ones that want to keep the current behaviour alias it to 'git cherry-pick -R'
-- ·O· Pierre Habouzit ··O madcoder@debian.org OOO http://www.madism.org