Re: [Qemu-devel] [PATCH RFC] MAINTAINERS: clarify some of the tags

[Markus Armbruster <armbru@redhat.com>]

Cornelia Huck <cohuck@redhat.com> writes:

> The MAINTAINERS file is a bit sparse on information about what
> the different designators are. Let's add some more information
> to give contributors a better idea about what the different
> roles are.
>
> Signed-off-by: Cornelia Huck <cohuck@redhat.com>
> ---
>
> This came out of a discussion about what being a 'reviewer' listed in
> this file actually means. A reviewer probably should already have a
> track record of doing helpful reviews before being listed in here.
>
> While at it, I also tried to add some more hints for the other entries.
> This patch is supposed to be a starting point for further discussion.
>
> ---
>  MAINTAINERS | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 40672c4eba..7d9dbc6724 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12,9 +12,14 @@ consult qemu-devel and not any specific individual privately.
>  Descriptions of section entries:
>  
>  	M: Mail patches to: FullName <address@domain>
> +	   Maintainers are looking after a certain area and must be CCed on
> +	   patches. They are considered the main contact point.
>  	R: Designated reviewer: FullName <address@domain>
>  	   These reviewers should be CCed on patches.
> +	   Reviewers are familiar with the subject matter and provide feedback
> +	   even though they are not maintainers.
>  	L: Mailing list that is relevant to this area
> +	   These lists should be CCed on patches.
>  	W: Web-page with status/info
>  	Q: Patchwork web based patch tracking system site
>  	T: SCM tree type and location.  Type is one of: git, hg, quilt, stgit.

Perfect is the enemy of good.  This is good.

Reviewed-by: Markus Armbruster <armbru@redhat.com>