Re: [Qemu-devel] [PATCH 1/5] MAINTAINERS: Add "R:" tag for self-appointed reviewers

[John Snow]

On 05/03/2017 05:43 AM, Paolo Bonzini wrote:
> From: "Paul E. McKenney" <address@hidden>
> 
> 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 <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
>  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@hidden>
> +     R: Designated reviewer: FullName <address@hidden>
> +        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
>