From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:46410)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1XgIC6-0008QZ-4N
	for qemu-devel@nongnu.org; Mon, 20 Oct 2014 15:00:39 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1XgIBw-0006uR-3s
	for qemu-devel@nongnu.org; Mon, 20 Oct 2014 15:00:37 -0400
Received: from mx1.redhat.com ([209.132.183.28]:63906)
	by eggs.gnu.org with esmtp (Exim 4.71)
	(envelope-from <mst@redhat.com>) id 1XgIBv-0006uG-AK
	for qemu-devel@nongnu.org; Mon, 20 Oct 2014 15:00:27 -0400
Date: Mon, 20 Oct 2014 22:03:58 +0300
From: "Michael S. Tsirkin" <mst@redhat.com>
Message-ID: <20141020190358.GA12441@redhat.com>
References: <1413796790-30579-1-git-send-email-armbru@redhat.com>
	<CAFEAcA_UxfVow0m2xu4+r8S-nK2Pb-jyNxmWhvXXP1G9V4Nerg@mail.gmail.com>
	<20141020141548.GA11062@redhat.com>
	<CAFEAcA_c2-qJ6PBEdLnCSFg6yy4EzXzy_17BHLoZQhwD=WGDyw@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAFEAcA_c2-qJ6PBEdLnCSFg6yy4EzXzy_17BHLoZQhwD=WGDyw@mail.gmail.com>
Subject: Re: [Qemu-devel] [PATCH] get_maintainer.pl: Default to
 --no-git-fallback
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Markus Armbruster <armbru@redhat.com>, QEMU Developers <qemu-devel@nongnu.org>

On Mon, Oct 20, 2014 at 03:19:52PM +0100, Peter Maydell wrote:
> On 20 October 2014 15:15, Michael S. Tsirkin <mst@redhat.com> wrote:
> > On Mon, Oct 20, 2014 at 03:04:44PM +0100, Peter Maydell wrote:
> >> On 20 October 2014 10:19, Markus Armbruster <armbru@redhat.com> wrote:
> >> > Contributors rely on this script to find maintainers to copy.  The
> >> > script falls back to git when no exact MAINTAINERS pattern matches.
> >> > When that happens, recent contributors get copied, which tends not be
> >> > particularly useful.  Some contributors find it even annoying.
> >> >
> >> > Flip the default to "don't fall back to git".  Use --git-fallback to
> >> > ask it to fall back to git.
> 
> >> Good idea.
> 
> > What do you want to happen in this case?
> 
> It should mail the people who are actually maintainers,
> not anybody who happened to touch the code in the last
> year.

Right but as often as not there's no data about that
in MAINTAINERS.


> > I'm yet to see contributors who are annoyed but we
> > can always blacklist specific people.
> 
> At the moment I just don't use get_maintainers.pl at
> all because I tried it a few times and it just cc'd
> a bunch of irrelevant people...
> 
> I suspect anybody using it at the moment is either
> using the --no-git-fallback flag or trimming the
> cc list a lot.
> 
> thanks
> -- PMM

I'm using it: sometimes with --no-git-fallback, sometimes without.

IIUC the default is to have up to 5 people on the Cc list
(--git-max-maintainers).
It's not like it adds 200 random people, is it?

Anyway experienced contributors can figure it out IMHO.


Question in my mind is what do we want a casual contributor
to do if there's no one listed in MAINTAINERS.
"Look in MAINTAINERS, if not there, look in git log"
sounds very reasonable to me, better than "CC no one".

-- 
MST