mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Joe Perches <joe@perches.com>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: Pekka Enberg <penberg@cs.helsinki.fi>,
	linux-kernel@vger.kernel.org,
	Andrew Morton <akpm@linux-foundation.org>,
	Vegard Nossum <vegard.nossum@gmail.com>
Subject: Re: [PATCH 5/7] MAINTAINERS: Remove L: linux-kernel@vger.kernel.org
Date: Fri, 10 Jul 2009 09:25:00 -0700	[thread overview]
Message-ID: <1247243100.11673.58.camel@Joe-Laptop.home> (raw)
In-Reply-To: <20090710131742.GB8878@rakim.wolfsonmicro.main>

On Fri, 2009-07-10 at 14:17 +0100, Mark Brown wrote:
> On Thu, Jul 09, 2009 at 12:35:39PM -0700, Joe Perches wrote:
> > On Thu, 2009-07-09 at 16:08 +0100, Mark Brown wrote:
> > > That's pretty much what I was thinking of too - tweaking the values for
> > > the log queries based on the total number of hits so people only turn up
> > > if they did a certain proportion of the commits.  It'd take some
> > > experimentation to work out what the values to use should be, I guess.
> > Perhaps something like this:
> > +    --git-min-percent => minimum percentage of commits required (default: 0)
> 
> That seems reasonably effective with 5% or so as the default.  I do
> think the default should be non-zero if we're going to be encouraging
> people to use this as standard, though.

I'm not sure minimization of maintainers in
the result is that desirable but I am for
figuring out what works best.

I set it at 0 for now just to play with it.

What have you experimented with and how have the
results changed?

If minimization is what's desired, what I've
tried seems to work best at around 10.

Here's the little script I used.

#!/bin/bash
#

ShowMaintainers() {
    file=$1
    percent=$2
    extraargs=$3

    echo
    echo "Maintainers for: $file @ $percent % $extraargs"
    echo
 
   ./scripts/get_maintainer.pl $extraargs --git-min-percent=$percent -f $file 
}

for dir in $(find drivers -type d) ; do
    dir=$(echo $dir | sed -e 's/\.\///g')
    ShowMaintainers $dir 0
    ShowMaintainers $dir 5
    ShowMaintainers $dir 10
    ShowMaintainers $dir 0 "--nogit"
done



  reply	other threads:[~2009-07-10 16:26 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-07-08  5:55 [PATCH 0/7] MAINTAINERS: cleanups Joe Perches
2009-07-08  5:55 ` [PATCH 1/7] MAINTAINERS: IA64 - pair P:/M: entries properly Joe Perches
2009-07-08  5:55 ` [PATCH 2/7] MAINTAINERS: Remove ivtv-user lists, add CX18 url Joe Perches
2009-07-08  5:55 ` [PATCH 3/7] MAINTAINERS: QLGE 10Gb ETHERNET - pair P:/M: entries properly Joe Perches
2009-07-08  5:55 ` [PATCH 4/7] MAINTAINERS: Use tabs in ACER ASPIRE ONE Joe Perches
2009-07-08  5:55 ` [PATCH 5/7] MAINTAINERS: Remove L: linux-kernel@vger.kernel.org Joe Perches
2009-07-08  6:32   ` Pekka Enberg
2009-07-08  6:39     ` Joe Perches
2009-07-08  6:44       ` Pekka Enberg
2009-07-08  6:47         ` Joe Perches
2009-07-08  6:55           ` Pekka Enberg
2009-07-08 16:37             ` Maciej W. Rozycki
2009-07-09  9:30           ` Mark Brown
2009-07-09 14:58             ` Joe Perches
2009-07-09 15:08               ` Mark Brown
2009-07-09 19:35                 ` Joe Perches
2009-07-10 13:17                   ` Mark Brown
2009-07-10 16:25                     ` Joe Perches [this message]
2009-07-10 18:44                       ` Mark Brown
2009-07-10 19:16                         ` Joe Perches
2009-07-08  5:55 ` [PATCH 6/7] MAINTAINERS: Move ARPD to CREDITS Joe Perches
2009-07-08  5:55 ` [PATCH 7/7] MAINTAINERS: Update KERNEL JANITORS Joe Perches
2009-07-09  2:25   ` Randy Dunlap

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=1247243100.11673.58.camel@Joe-Laptop.home \
    --to=joe@perches.com \
    --cc=akpm@linux-foundation.org \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=penberg@cs.helsinki.fi \
    --cc=vegard.nossum@gmail.com \
    /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

all inboxes | Powered by JetHome®