linux-kernel - Re: [PATCH] get_maintainer.pl: ignore non-maintainer tags

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <4BE46E7D.1040906@s5r6.in-berlin.de>
Date:	Fri, 07 May 2010 21:48:13 +0200
From:	Stefan Richter <stefanr@...6.in-berlin.de>
To:	Florian Mickler <florian@...kler.org>
CC:	Joe Perches <joe@...ches.com>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Stephen Hemminger <shemminger@...tta.com>,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] get_maintainer.pl: ignore non-maintainer tags

Florian Mickler wrote:
> On Thu, May 06, 2010 at 11:39:26PM -0700, Joe Perches wrote:
>> On Fri, 2010-05-07 at 08:34 +0200, florian@...kler.org wrote:
>>> Using --git to determine who to send a patch to, it is not
>>> reasonable to include people that only reported an issue or tested a
>>> patch.
>> I think this is a questionable assumption.
>>
>> People that test or otherwise sign a patch are also good
>> candidates to review new patches.
>>
>> cheers, Joe
> 
> Our views differ then.
> 
> It is unreasonable to assume, that someone who is listed via
> Tested-By: in a random patch is able to review another patch to the same
> file. More likely will he be annoyed, because he see's a patch, dig's
> in and then thinks: "wtf? why were i cc'ed on this. I don't know this
> code. I don't have time for this!"
> 
> Thats just broken.

Indeed.  Three points:

  - Reported-by: and Tested-by: signed commits were most likely about
    something radically different from what the new patch submission is
    about.

  - get_maintainer.pl is supposed to list maintainer addresses.
    Reporters and testers are not maintainers.

  - If a get_maintainer.pl user expects to get not only maintainer
    addresses but also addresses of potential reviewers, then he shall
    use the mailinglist addresses for that.
    People who /want/ to review patches subscribe to that mailinglist
    or register themselves in MAINTAINERS.

Pushing patches to people of whom it is unknown whether they want or can
review the patches may actually be rude.  If it can be avoided that
get_maintainer.pl lists addresses of people who are likely of that
category, then please teach it to.
-- 
Stefan Richter
-=====-==-=- -=-= --===
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/