Messages in this thread | | | Date | Fri, 7 May 2010 09:02:18 +0200 | From | Florian Mickler <> | Subject | Re: [PATCH] get_maintainer.pl: ignore non-maintainer tags |
| |
On Thu, May 06, 2010 at 11:39:26PM -0700, Joe Perches wrote: > On Fri, 2010-05-07 at 08:34 +0200, florian@mickler.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.
cheers, Flo
| |