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

[Markus Armbruster]

Cornelia Huck <address@hidden> 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 <address@hidden>
> ---
>
> 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@hidden>
> +        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@hidden>
>          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 <address@hidden>