Re: [Qemu-devel] [PATCH] MAINTAINERS: add all-match entry for qemu-devel@

[Markus Armbruster]

Stephen Warren <address@hidden> writes:

> Add an entry to MAINTAINERS that matches every patch, and requests the
> user send patches to address@hidden
>
> It's not 100% obvious to project newcomers that all patches should be sent
> there; checkpatch doesn't say so, and since it mentions other lists to CC,
> the wording "the list" from the SubmitAPatch wiki page can be taken
> to mean only those lists, not the main list too.
>
> The F: entries were taken from a similar entry in the Linux kernel.
>
> Signed-off-by: Stephen Warren <address@hidden>
> ---
>  MAINTAINERS | 5 +++++
>  1 file changed, 5 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index b6ed87a26349..2d78eea5d62d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -52,6 +52,11 @@ General Project Administration
>  ------------------------------
>  M: Peter Maydell <address@hidden>
>  
> +All patches CC here
> +L: address@hidden
> +F: *
> +F: */
> +
>  Responsible Disclosure, Reporting Security Issues
>  ------------------------------
>  W: http://wiki.qemu.org/SecurityProcess

I acknowledge the problem, but your patch breaks the "No maintainers
found, printing recent contributors" functionality.

Before:

    $ scripts/get_maintainer.pl -f MAINTAINERS 
    get_maintainer.pl: No maintainers found, printing recent contributors.
    get_maintainer.pl: Do not blindly cc: them on patches!  Use common sense.

    Paolo Bonzini <address@hidden> (commit_signer:20/74=27%)
    Kevin Wolf <address@hidden> (commit_signer:13/74=18%)
    Stefan Hajnoczi <address@hidden> (commit_signer:13/74=18%)
    "Michael S. Tsirkin" <address@hidden> (commit_signer:10/74=14%)
    Peter Maydell <address@hidden> (commit_signer:8/74=11%)

After:

    $ scripts/get_maintainer.pl -f MAINTAINERS 
    address@hidden (open list:All patches CC here)