git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] shortlog: take the first populated line of the description

From
Nicolas Pitre <nico@cam.org>
Date
Mar 5, 2008, 18:47 UTC
Message-ID
<alpine.LFD.1.00.0803051346200.2947@xanadu.home>
In-Reply-To
<1204727050.0@pinky>
On Wed, 5 Mar 2008, Andy Whitcroft wrote:
Show 20 quoted lines
> 
> Way back the perl version of shortlog would take the first populated line
> of the commit body.  The builtin version mearly takes the first line.
> This leads to empty shortlog entries when there is some viable text in
> the commit.
> 
> Reinstate this behaviour igoring all lines with nothing but whitespace.
> 
> Signed-off-by: Andy Whitcroft <apw@shadowen.org>
> ---
> 
> 	This seems to be an improvement, returning to the original
> 	behaviour.  I cannot think of any good reason not to take the first
> 	populated line for a shortlog.	The alternative less agressive
> 	compromise might be to skip only completly empty lines at the
> 	start, but I am not sure that adds any value.
> 
> 	I seem to get a lot of these in converted SVN commits.
> 
> 	Comments?
Maybe it is the SVN conversion that needs fixing?
Nicolas
Previous: Johannes SchindelinNext: Junio C Hamano
Message 5 of 8 in “shortlog: take the first populated line of the description”
  1. shortlog: take the first populated line of the descriptionAndy Whitcroft, Mar 5, 2008
  2. Johannes SchindelinMar 5, 2008
  3. Andy WhitcroftMar 5, 2008
  4. Johannes SchindelinMar 5, 2008
  5. Nicolas PitreMar 5, 2008
  6. Junio C HamanoMar 5, 2008
  7. Nicolas PitreMar 5, 2008
  8. Junio C HamanoMar 5, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.