On Tue, 2014-06-03 at 14:48 -0700, Paul E. McKenney wrote: > On Tue, Jun 03, 2014 at 01:16:07PM -0700, Joe Perches wrote: > > On Tue, 2014-06-03 at 13:12 -0700, Paul E. McKenney wrote: > > > On Tue, Jun 03, 2014 at 12:03:07PM -0700, Joe Perches wrote: > > > > On Tue, 2014-06-03 at 08:35 -0700, Paul E. McKenney wrote: > > > > > Paul Walmsley also noted the need for patch > > > > > submitters to know who the key reviewers are and suggested adding an > > > > > "R:" tag to the MAINTAINERS file to record this information on a > > > > > per-subsystem basis. > > > > [] > > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > > > [] > > > > > @@ -70,6 +70,8 @@ Descriptions of section entries: > > > > > > > > > > P: Person (obsolete) > > > > > M: Mail patches to: FullName <address@domain> > > > > > + R: Designated reviewer, who should be CCed on patches, > > > > > + format: FullName <address@domain> > > > > > L: Mailing list that is relevant to this area > > > > > W: Web-page with status/info > > > > > Q: Patchwork web based patch tracking system site > > > > > > > > Perhaps the M: and R: roles should be more fully > > > > described here. > > > > > > > > The R: entry description doesn't need commas. > > > > > > > > Maybe something like: > > > > M: Maintainer (to:) FullName <address@domain> > > > > R: Reviewer (cc:) FullName <address@domain> > > > > > > This is arguably better organized, but I need to keep my focus on adding > > > the designated reviewer. I am not in a position to undertake a rewrite > > > of the MAINTAINER file preamble at this point. > > > > That's silly, you're adding the "R" entry. > > Add a proper description for it. > > I will take another look at R: in a few days and possibly wordsmith it.
Swell, thanks. > My reluctance is due to the fact that this will change how newbies > submit patches. Very, very doubtful that will change. > But if we get a few Acked-by or Reviewed-by responses > from high-level maintainers (Linus, Greg, Ingo, Thomas, Peter, Ted, > Dave, Ben, ...), I will of course be happy to queue it. cheers, Joe -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
