Message ID | 20170503094351.5040-2-pbonzini@redhat.com |
---|---|
State | New |
Headers | show |
On 05/03/2017 05:43 AM, Paolo Bonzini wrote: > From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com> > > Some people are not content with the amount of mail they get, and would > like to be CCed on patches for areas they do not maintain. Let them > satisfy their own appetite for qemu-devel messages. > > Seriously: the purpose here is a bit different from the Linux kernel. > While Linux uses "R" to designate non-maintainers for reviewing patches > in a given area, in QEMU I would also like to use "R" so that people can > delegate sending pull requests while keeping some degree of oversight. > Hm? I don't follow -- you mean to say that "R" is also a designated pull-request-sender? Did you document that anywhere? --js > Based on Linux commit eafbaac3093760d1fd3b2a5b9f016362dd68af36. > > Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com> > Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> > --- > MAINTAINERS | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index a1d2b3a4d3..46cf4870f9 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -12,6 +12,8 @@ consult qemu-devel and not any specific individual privately. > Descriptions of section entries: > > M: Mail patches to: FullName <address@domain> > + R: Designated reviewer: FullName <address@domain> > + These reviewers should be CCed on patches. > L: Mailing list that is relevant to this area > W: Web-page with status/info > Q: Patchwork web based patch tracking system site >
On 03/05/2017 20:23, John Snow wrote: >> >> Seriously: the purpose here is a bit different from the Linux kernel. >> While Linux uses "R" to designate non-maintainers for reviewing patches >> in a given area, in QEMU I would also like to use "R" so that people can >> delegate sending pull requests while keeping some degree of oversight. >> > Hm? I don't follow -- you mean to say that "R" is also a designated > pull-request-sender? Did you document that anywhere? No, "M" would be the pull request sender, while "R" would be me delegating while keeping an eye on what's going on. Paolo
diff --git a/MAINTAINERS b/MAINTAINERS index a1d2b3a4d3..46cf4870f9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -12,6 +12,8 @@ consult qemu-devel and not any specific individual privately. Descriptions of section entries: M: Mail patches to: FullName <address@domain> + R: Designated reviewer: FullName <address@domain> + These reviewers should be CCed on patches. L: Mailing list that is relevant to this area W: Web-page with status/info Q: Patchwork web based patch tracking system site