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

From: Florian Mickler
Date: Fri May 07 2010 - 03:34:20 EST


On Thu, May 06, 2010 at 11:39:26PM -0700, Joe Perches wrote:
> On Fri, 2010-05-07 at 08:34 +0200, florian@xxxxxxxxxxx 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.

cheers,
Flo


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/