[lkml]   [2010]   [May]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] 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, 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

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

    - If a 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 lists addresses of people who are likely of that
    category, then please teach it to.
    Stefan Richter
    -=====-==-=- -=-= --===

     \ /
      Last update: 2010-05-07 21:51    [W:0.023 / U:128.024 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site