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

[Stephen Warren]

On 02/08/2016 11:52 AM, John Snow wrote:
> On 02/08/2016 04:05 AM, Markus Armbruster wrote:
> > 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)
>
> Agree ... Might this also break any "unmaintained" scripts we use?
>
> How about:
>   1) Update the wiki to be clearer on the submission process, and
>   2) Update the "no contributors found" output to also include the list
> to send to?

My case was that maintainers were found, but the list only included the 
maintainer (Peter) and the ARM list, but not the main qemu list. So (2) 
above wouldn't have solved my original issue.

qemu's get_maintainers.pl must be doing something different to the Linux 
kernel's version then, since IIRC the Linux kernel version always prints 
out recent contributors irrespective of whether maintainers were found, 
rather than as an alternative. I think if qemu's get_maintainers.pl 
acted in that way it would solve the problem my patch caused?