Git development
 help / color / mirror / Atom feed
From: Junio C Hamano <junkio@cox.net>
To: Carl Worth <cworth@cworth.org>
Cc: git@vger.kernel.org
Subject: Re: [PATCH] Munge ChangeLog-style commit messages, and grab author name and email.
Date: Fri, 10 Feb 2006 21:39:16 -0800	[thread overview]
Message-ID: <7v3biqpi7v.fsf@assigned-by-dhcp.cox.net> (raw)
In-Reply-To: <87slqqvgj2.wl%cworth@cworth.org> (Carl Worth's message of "Fri, 10 Feb 2006 17:19:13 -0800")

Carl Worth <cworth@cworth.org> writes:

> The cairo project has been using ChangeLog-style commit messages, such as:
>
> 	2006-01-18  Carl Worth  <cworth@cworth.org>
>
> 		* src/cairo-pdf-surface.c: (cairo_pdf_surface_create_for_stream),
> 		(cairo_pdf_surface_create): Fix compilation-breaking typo.
>
> This patch recognizes that style and munges it up into:
>
> 	Fix compilation-breaking typo.
>
> which results in a much more useful headline within git.

I welcome discussion on this patch, but as its in current shape,
it looks a bit too specific to one particular style.

I think the one you quoted above is the GNU official style which
is used in many projects.  I suspect your patch would help
people converting other projects as well.  Having said that, as
you said it yourself, it would make more sense to make a more
generalized log munging interface.

Even though there might be many different schools of style,
hopefully each project consistently sticks to one style.  Log
messages out of SVN repository tend to have their own styles.
We could define a common command line option, say --log-filter,
that are available across importers.  E.g.

	git-cvsimport --log-filter=gnu-log-style
	git-svnimport --log-filter=gnu-log-style

'gnu-log-style' would be the massager you have hardcoded in the
patch, but made into a traditional UNIXy filter.  We could
create contrib/commit-filters directory in git.git repository
and ship collections of such log massagers.

      reply	other threads:[~2006-02-11  5:39 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-02-11  1:19 [PATCH] Munge ChangeLog-style commit messages, and grab author name and email Carl Worth
2006-02-11  5:39 ` Junio C Hamano [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=7v3biqpi7v.fsf@assigned-by-dhcp.cox.net \
    --to=junkio@cox.net \
    --cc=cworth@cworth.org \
    --cc=git@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox