Re: [PATCH] Teach the git wrapper about --name-rev and --name-rev-by-tags

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Jul 28, 2006, 18:53 UTC
Message-ID: <Pine.LNX.4.63.0607282042470.29667@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To: <Pine.LNX.4.64.0607280952200.4168@g5.osdl.org>

Hi,

On Fri, 28 Jul 2006, Linus Torvalds wrote:

Show 7 quoted lines

> On Fri, 28 Jul 2006, Johannes Schindelin wrote:
> > 
> > Now you can say
> > 
> > 	git --name-rev log
> 
> I think this is wrong.

I think it is not wrong. :-)

Show 5 quoted lines

> It may be a straightforward translation of
> 
> > 	git log | git name-rev --stdin | less
> 
> but that doesn't make it any more "correct".

I use it also for other git commands, so this was very much on purpose.

> Also, I doubt most people want every release named.

You are probably right. But _I_ want to know that e.g. commit a025463bc0ec2c894a88f2dfb44cf88ba71bb712 is really tags/v1.4.0^0~27^2. Both are immutable, but the latter is nicer to people than to computers.

Show 7 quoted lines

> I think the common case would be that you want those releases named that 
> match heads (and tags in particular) _exactly_. If you want everything 
> named, maybe you want to do "--name-rev-all" or something.
> 
> Hmm?
> 
> (That would also likely perform a lot better)

True. But then, you probably know which head it is, because you probably specified it yourself on the command line.

Ciao, Dscho