Re: commit summary, --pretty=short and other tools

From: Andreas Ericsson <ae@op5.se>
Date: Sep 18, 2007, 07:19 UTC
Message-ID: <46EF7BF7.3070107@op5.se>
In-Reply-To: <55887C88-8523-4839-8B91-236256A5E893@lrde.epita.fr>

Benoit SIGOURE wrote:

> 
> My opinion is that it would be better to keep the first line and never 
> ever rewrite the commit messages.
>

I've had reason to ponder this quite a lot, as I've imported 15 repos from CVS and SVN where the commit authors did not follow the git-recommended way of doing things, but rather put everything as one paragraph, usually without linebreaks, in the commit message.

>From what I've read from those rather horrid commit-messages so far, it's

usually correct to grab the first sentence in case the empty line isn't there, so:

const char *find_commit_subject_end(const char *commit_msg)
{
	const char *dot, *paragraph_end;
	
	paragraph_end = strstr(commit_msg, "\n\n");
	dot = strchr(commit_msg, '.');
	
	return min_non_null(dot, paragraph_end); 
}

would probably get it right very nearly always.

I'll submit a patch in 3 hours when I get my lunch, unless someone beats me to it.

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231