LKML Archive on lore.kernel.org
 help / color / Atom feed
From: Joe Perches <joe@perches.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Jiri Kosina <trivial@kernel.org>,
	Andrew Morton <akpm@linux-foundation.org>,
	Florian Mickler <florian@mickler.org>,
	alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: rfc: rewrite commit subject line for subsystem maintainer preference tool
Date: Tue, 16 Nov 2010 06:51:17 -0800
Message-ID: <1289919077.28741.50.camel@Joe-Laptop> (raw)
In-Reply-To: <20101116104921.GL12986@rakim.wolfsonmicro.main>

On Tue, 2010-11-16 at 10:49 +0000, Mark Brown wrote:
> On Mon, Nov 15, 2010 at 11:52:53AM -0800, Joe Perches wrote:
> > On Mon, 2010-11-15 at 19:34 +0000, Mark Brown wrote:
> > > It appears your scripts are already hooked into get_maintainers.pl which
> > > would seem the obvious place to do this?  Sadly I don't do perl, though
> > > it looks like you're doing pretty much all the work on that anyway.
> > Sadly, no it's not the right place.
> To query MAINTAINERS?  I'd assume that's where you'd want to put that
> stuff?

I trimmed cc's and added Andrew Morton and Florian Mickler.
First thread link for them: http://lkml.org/lkml/2010/11/15/262

I use get_maintainer to find email addresses with
"git send-email --cc-cmd=" but sure it could be extended
to find some other new information in the MAINTAINERS file.

Anyway, I think that get_maintainers isn't the proper tool
to rewrite commit subject lines, though it could certainly
do the lookup of a key in the MAINTAINERS file.

Maybe add a new MAINTAINERS section line something like:
	"C:	CommitSubjectGrammarStyle"
where CommitSubjectGrammarStyle is something more
information rich than "style 1", "style 2".

Perhaps you'll propose a grammar to convert path to header
and go through and add these "C:" style entries to the
sections you maintain.

Also, what would you expect the output to be when a single
patch modified files from 2 subsystems that use different
styles?

cheers, Joe


  reply index

Thread overview: 100+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-15  3:04 [PATCH 00/44] remove unnecessary semicolons Joe Perches
2010-11-15  3:04 ` [PATCH 01/44] arch/arm: Remove " Joe Perches
2010-11-15  3:04 ` [PATCH 02/44] arch/microblaze: " Joe Perches
2010-11-15  6:37   ` Michal Simek
2010-11-15  3:04 ` [PATCH 03/44] arch/um: " Joe Perches
2010-11-15  3:04 ` [PATCH 04/44] drivers/cpufreq: " Joe Perches
2010-11-15  3:04 ` [PATCH 05/44] drivers/gpio: " Joe Perches
2010-11-15  3:04 ` [PATCH 06/44] drivers/i2c: " Joe Perches
2010-11-15  3:04 ` [PATCH 07/44] drivers/isdn: " Joe Perches
2010-11-15 19:08   ` David Miller
2010-11-15  3:04 ` [PATCH 08/44] drivers/leds: " Joe Perches
2010-11-15  3:04 ` [PATCH 09/44] drivers/media/video: " Joe Perches
2010-11-15  3:04 ` [PATCH 10/44] drivers/misc: " Joe Perches
2010-11-15  3:04 ` [PATCH 11/44] drivers/mmc: " Joe Perches
2010-12-05  3:32   ` Chris Ball
2010-11-15  3:04 ` [PATCH 12/44] drivers/net/bnx2x: " Joe Perches
2010-11-15 19:08   ` David Miller
2010-11-15  3:04 ` [PATCH 13/44] drivers/net/e1000e: " Joe Perches
2010-11-15 19:08   ` David Miller
2010-11-15  3:04 ` [PATCH 14/44] drivers/net/ixgbe: " Joe Perches
2010-11-15 16:24   ` Rose, Gregory V
2010-11-15 19:08   ` David Miller
2010-11-15  3:04 ` [PATCH 15/44] drivers/net/vxge: " Joe Perches
2010-11-15 19:07   ` David Miller
2010-11-15  3:04 ` [PATCH 16/44] drivers/net/wireless/ath: " Joe Perches
2010-11-15  3:04 ` [PATCH 17/44] drivers/net/wireless/iwlwifi: " Joe Perches
2010-11-15  3:04 ` [PATCH 18/44] drivers/net/cnic.c: " Joe Perches
2010-11-15 19:08   ` David Miller
2010-11-15  3:04 ` [PATCH 19/44] drivers/platform/x86: " Joe Perches
2010-11-24 16:52   ` Matthew Garrett
2010-11-15  3:04 ` [PATCH 20/44] drivers/power: " Joe Perches
2010-11-15  3:04 ` [PATCH 21/44] drivers/s390/net: " Joe Perches
2010-11-15 19:08   ` David Miller
2010-11-15  3:04 ` [PATCH 22/44] drivers/scsi/be2iscsi: " Joe Perches
2010-11-15  3:04 ` [PATCH 23/44] drivers/scsi/bfa: " Joe Perches
2010-11-15  3:04 ` [PATCH 24/44] drivers/scsi/lpfc: " Joe Perches
2010-11-15  3:04 ` [PATCH 25/44] drivers/scsi/pm8001: " Joe Perches
2010-11-15  3:27   ` Jack Wang
2010-11-15  3:04 ` [PATCH 26/44] drivers/scsi/qla2xxx: " Joe Perches
2010-11-15  3:04 ` [PATCH 27/44] drivers/serial: " Joe Perches
2010-11-15  3:04 ` [PATCH 28/44] drivers/spi: " Joe Perches
2010-11-15  3:57   ` Grant Likely
2010-11-15  3:04 ` [PATCH 29/44] drivers/staging: " Joe Perches
2010-11-15  3:04 ` [PATCH 30/44] drivers/usb/gadget: " Joe Perches
2010-11-15  3:04 ` [PATCH 31/44] drivers/xen: " Joe Perches
2010-11-15  3:04 ` [PATCH 32/44] fs/9p: " Joe Perches
2010-11-15  3:04 ` [PATCH 33/44] fs/ceph: " Joe Perches
2010-11-15  3:04 ` [PATCH 34/44] fs/logfs: " Joe Perches
2010-11-15  3:04 ` [PATCH 35/44] fs/nfs: " Joe Perches
2010-11-15  3:04 ` [PATCH 36/44] fs/ocfs2: " Joe Perches
2010-11-17  0:10   ` Joel Becker
2010-11-15  3:04 ` [PATCH 37/44] fs/ubifs: " Joe Perches
2010-11-19 13:21   ` Artem Bityutskiy
2010-11-15  3:04 ` [PATCH 38/44] include/linux/if_macvlan.h: " Joe Perches
2010-11-15 19:07   ` David Miller
2010-11-15  3:04 ` [PATCH 39/44] include/net/caif/cfctrl.h: " Joe Perches
2010-11-15  7:12   ` Sjur BRENDELAND
2010-11-15 19:07   ` David Miller
2010-11-15  3:04 ` [PATCH 40/44] mm/hugetlb.c: " Joe Perches
2010-11-15  9:52   ` Mel Gorman
2010-11-15  3:05 ` [PATCH 41/44] net/ipv6/mcast.c: " Joe Perches
2010-11-15 19:07   ` David Miller
2010-11-15  3:05 ` [PATCH 42/44] net/sunrpc/addr.c: " Joe Perches
2010-11-15  3:05 ` [PATCH 43/44] sound/core/pcm_lib.c: " Joe Perches
2010-11-22  6:44   ` Takashi Iwai
2010-11-15  3:05 ` [PATCH 44/44] sound/soc/codecs: " Joe Perches
2010-11-15 11:09   ` Liam Girdwood
2010-11-15 13:49   ` Mark Brown
2010-11-15 17:09     ` Joe Perches
2010-11-15 17:30       ` Mark Brown
2010-11-15 17:34         ` Joe Perches
2010-11-15 18:27           ` Mark Brown
2010-11-15 18:30             ` Joe Perches
2010-11-15 19:07               ` Mark Brown
2010-11-15 19:14                 ` Joe Perches
2010-11-15 19:34                   ` Mark Brown
2010-11-15 19:52                     ` Joe Perches
2010-11-16 10:49                       ` Mark Brown
2010-11-16 14:51                         ` Joe Perches [this message]
2010-11-16 15:04                           ` rfc: rewrite commit subject line for subsystem maintainer preference tool Mark Brown
2010-11-16 17:37                           ` Florian Mickler
2010-11-16 18:12                             ` Mark Brown
2010-11-16 19:35                               ` Florian Mickler
2010-11-16 19:55                                 ` Mark Brown
2010-11-16 20:21                                   ` Randy Dunlap
2010-11-16 20:42                                     ` Joe Perches
2010-11-16 20:46                                       ` Randy Dunlap
2010-11-16 23:22                                         ` Mark Brown
2010-11-16 23:28                                           ` Randy Dunlap
2010-11-16 23:50                                             ` Mark Brown
2010-11-16 23:57                                             ` Joe Perches
2010-11-17  0:55                                               ` Randy Dunlap
2010-11-16 23:57                                           ` Joe Perches
2010-11-16 23:01                                     ` Mark Brown
2010-11-17  0:44                                       ` Stefan Richter
2010-11-17  0:53                                         ` Jiri Kosina
2010-11-17 17:07                                           ` Mark Brown
2010-11-17 21:07                                             ` Stefan Richter
2010-11-17 23:49                                               ` Joe Perches
2010-11-17 19:32           ` [PATCH 44/44] sound/soc/codecs: Remove unnecessary semicolons Pavel Machek

Reply instructions:

You may reply publically 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=1289919077.28741.50.camel@Joe-Laptop \
    --to=joe@perches.com \
    --cc=akpm@linux-foundation.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=florian@mickler.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=trivial@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

LKML Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/lkml/0 lkml/git/0.git
	git clone --mirror https://lore.kernel.org/lkml/1 lkml/git/1.git
	git clone --mirror https://lore.kernel.org/lkml/2 lkml/git/2.git
	git clone --mirror https://lore.kernel.org/lkml/3 lkml/git/3.git
	git clone --mirror https://lore.kernel.org/lkml/4 lkml/git/4.git
	git clone --mirror https://lore.kernel.org/lkml/5 lkml/git/5.git
	git clone --mirror https://lore.kernel.org/lkml/6 lkml/git/6.git
	git clone --mirror https://lore.kernel.org/lkml/7 lkml/git/7.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 lkml lkml/ https://lore.kernel.org/lkml \
		linux-kernel@vger.kernel.org
	public-inbox-index lkml

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-kernel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git