Re: [PATCH] [RFC] get_maintainer.pl: only list maintainers by default

[Florian Mickler <florian@mickler.org>]

On Sat, 08 May 2010 15:44:34 -0700
Joe Perches <joe@perches.com> wrote:

> On Sat, 2010-05-08 at 23:39 +0200, florian@mickler.org wrote:
> > If scm meta-information is used, only list people that signed-off or
> > acked or reviewed the code in question.
> > 
> > after all it's called get_maintainer.pl and not
> > get_random_people_to_look_at_the_code.pl
> 
> Anyone that actually signs a patch is not "random".
> 

I actually think that tested-by and reviewed-by is not "signing",
because it is not the person who does the "tag insertion". 

it was one of the signed-off-by's who put the reviewed- and testet-tags
in there. presumably this is done in agreement with the mentioned
person. 

And for facts, please check this out:

$ git log --since=1-year-ago | grep -Pi "(reported|tested)-by:.*@"  \
 | cut -f2- -d":" | sort | uniq | wc -l
1280

so i would say, these numbers (combied with your top ten which
accounts for only 318 signatures ) suggest that those mentioned with
these tag's are one-off bug reporters and not maintainers.

cheers,
Flo