Re: [PATCH] shortlog: prompt when reading from terminal by mistake

From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Mar 8, 2007, 17:37 UTC
Message-ID: <Pine.LNX.4.63.0703081834310.22628@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To: <Pine.LNX.4.64.0703080928580.10832@woody.linux-foundation.org>

Hi,

On Thu, 8 Mar 2007, Linus Torvalds wrote:

Show 5 quoted lines

> On Thu, 8 Mar 2007, Johannes Schindelin wrote:
> > 
> > How about something totally different: "git log --shortlog". I.e.
> 
> Sure, but this is even worse, because the way you wrote the patch

I should have marked it as "this is the idea, but even if it looks like a patch, it is not".

Unfortunately, I don't have time to do a proper patch, which is why I just wrote that simple and completely untested example of how this could be done.

I'd split off a proper builtin_shortlog(revs) from cmd_shortlog(), to avoid double parsing, and also because it looks nicer to the eye, but time is scarce.

Ciao, Dscho