git.notmuchmail.org Git - notmuch/blob - test/corpora/lkml/cur/1382298793.003216:2,

   1 From: Florian Mickler <florian@mickler.org>
   2 Subject: Re: rfc: rewrite commit subject line for subsystem maintainer
   3  preference tool
   4 Date: Tue, 16 Nov 2010 18:37:07 +0100
   5 Lines: 59
   6 Message-ID: <20101116183707.179964dd@schatten.dmk.lab>
   7 References: <20101115134939.GC12986@rakim.wolfsonmicro.main>
   8         <1289840957.16461.138.camel@Joe-Laptop>
   9         <20101115173031.GI12986@rakim.wolfsonmicro.main>
  10         <1289842444.16461.140.camel@Joe-Laptop>
  11         <20101115182708.GJ12986@rakim.wolfsonmicro.main>
  12         <1289845830.16461.149.camel@Joe-Laptop>
  13         <20101115190738.GF3338@sirena.org.uk>
  14         <1289848458.16461.150.camel@Joe-Laptop>
  15         <20101115193407.GK12986@rakim.wolfsonmicro.main>
  16         <1289850773.16461.166.camel@Joe-Laptop>
  17         <20101116104921.GL12986@rakim.wolfsonmicro.main>
  18         <1289919077.28741.50.camel@Joe-Laptop>
  19 Mime-Version: 1.0
  20 Content-Type: text/plain; charset=US-ASCII
  21 Content-Transfer-Encoding: 7bit
  22 Cc: Mark Brown <broonie@opensource.wolfsonmicro.com>,
  23         Jiri Kosina <trivial@kernel.org>,
  24         Andrew Morton <akpm@linux-foundation.org>,
  25         alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
  26 To: Joe Perches <joe@perches.com>
  27 X-From: linux-kernel-owner@vger.kernel.org Tue Nov 16 18:37:57 2010
  28 Return-path: <linux-kernel-owner@vger.kernel.org>
  29 Envelope-to: glk-linux-kernel-3@lo.gmane.org
  30 Received: from vger.kernel.org ([209.132.180.67])
  31         by lo.gmane.org with esmtp (Exim 4.69)
  32         (envelope-from <linux-kernel-owner@vger.kernel.org>)
  33         id 1PIPTh-0007Ey-5Q
  34         for glk-linux-kernel-3@lo.gmane.org; Tue, 16 Nov 2010 18:37:57 +0100
  35 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
  36         id S1756054Ab0KPRhi (ORCPT <rfc822;glk-linux-kernel-3@m.gmane.org>);
  37         Tue, 16 Nov 2010 12:37:38 -0500
  38 Received: from ist.d-labs.de ([213.239.218.44]:44291 "EHLO mx01.d-labs.de"
  39         rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
  40         id S1754101Ab0KPRhh (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
  41         Tue, 16 Nov 2010 12:37:37 -0500
  42 Received: from schatten.dmk.lab (f053209081.adsl.alicedsl.de [78.53.209.81])
  43         by mx01.d-labs.de (Postfix) with ESMTPSA id 1EB9E7FFD4;
  44         Tue, 16 Nov 2010 18:36:55 +0100 (CET)
  45 In-Reply-To: <1289919077.28741.50.camel@Joe-Laptop>
  46 X-Mailer: Claws Mail 3.7.6cvs31 (GTK+ 2.20.1; x86_64-unknown-linux-gnu)
  47 Sender: linux-kernel-owner@vger.kernel.org
  48 Precedence: bulk
  49 List-ID: <linux-kernel.vger.kernel.org>
  50 X-Mailing-List: linux-kernel@vger.kernel.org
  51 Archived-At: <http://permalink.gmane.org/gmane.linux.kernel/1063246>
  52
  53 On Tue, 16 Nov 2010 06:51:17 -0800
  54 Joe Perches <joe@perches.com> wrote:
  55
  56 > On Tue, 2010-11-16 at 10:49 +0000, Mark Brown wrote:
  57 > > On Mon, Nov 15, 2010 at 11:52:53AM -0800, Joe Perches wrote:
  58 > > > On Mon, 2010-11-15 at 19:34 +0000, Mark Brown wrote:
  59 > > > > It appears your scripts are already hooked into get_maintainers.pl which
  60 > > > > would seem the obvious place to do this?  Sadly I don't do perl, though
  61 > > > > it looks like you're doing pretty much all the work on that anyway.
  62 > > > Sadly, no it's not the right place.
  63 > > To query MAINTAINERS?  I'd assume that's where you'd want to put that
  64 > > stuff?
  65 >
  66 > I trimmed cc's and added Andrew Morton and Florian Mickler.
  67 > First thread link for them: http://lkml.org/lkml/2010/11/15/262
  68 >
  69 > I use get_maintainer to find email addresses with
  70 > "git send-email --cc-cmd=" but sure it could be extended
  71 > to find some other new information in the MAINTAINERS file.
  72 >
  73 > Anyway, I think that get_maintainers isn't the proper tool
  74 > to rewrite commit subject lines, though it could certainly
  75 > do the lookup of a key in the MAINTAINERS file.
  76 >
  77 > Maybe add a new MAINTAINERS section line something like:
  78 >       "C:     CommitSubjectGrammarStyle"
  79 > where CommitSubjectGrammarStyle is something more
  80 > information rich than "style 1", "style 2".
  81 >
  82 > Perhaps you'll propose a grammar to convert path to header
  83 > and go through and add these "C:" style entries to the
  84 > sections you maintain.
  85 >
  86 > Also, what would you expect the output to be when a single
  87 > patch modified files from 2 subsystems that use different
  88 > styles?
  89 >
  90 > cheers, Joe
  91 >
  92
  93 My first reaction to this is, it's silly. Certainly a
  94 subsystem-maintainer is capable of hacking something together that
  95 suits his needs or may just use a good editor to get the job done.
  96 After all, he might want to edit the commit message anyway. Also he has
  97 to have his act together for all non-conforming submitters anyway,
  98 because shurely, telling people to re-edit their patches subject line
  99 is not what one would consider "welcoming to newbies",  or whatever it
 100 is kernel subsystem maintainers have to be nowadays *g*...
 101
 102 On second thought, if that facility existed, i think nobody would mind
 103 it either. So, why not. I don't see a way to specify what to do with
 104 cross-subsystem patches though.
 105
 106 (MAINTAINERS seems to be the logical place to put this
 107 information.)
 108
 109 Regards,
 110 Flo
 111
 112
 113