Thread (1 message) 1 message, 1 author, 2016-06-15

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

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:44:20

Nicolas Pitre [off-list ref] writes:
On Wed, 5 Mar 2008, Junio C Hamano wrote:
quoted
I think Andy's fix to make the output side take away
unnecessary blank lines is unconditionally good.
It is not "bad" in itself.  However that feels like papering over 
another problem which IMHO has greater merits to be fixed.  We have 
given special semantics to the first line of a commit log in many other 
places now, so unless all those places are also made aware of 
substandard commit logs too, I think it would be more productive to make 
sure those logs are semi sensible upon entering Git in the first place 
instead.
My understanding of the current status when I wrote that message
was that everybody at commit.c layer (that is, everybody except
shortlog) strips such garbage.  Do you have a specific broken
one in mind?

I suspect --pretty=format:%s might be broken; I didn't check.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help