Re: [Qemu-devel] [PATCH] MAINTAINERS: Fix subsystem name for "Build and test automation"

[Fam Zheng <famz@redhat.com>]

On Fri, 09/22 05:14, Thomas Huth wrote:
> On 21.09.2017 19:02, Eduardo Habkost wrote:
> > The subsystem name for the "Build test automation" section is
> > "-------------------------", because an actual subsystem name
> > line is missing:
> > 
> >   $ ./scripts/get_maintainer.pl -f tests/docker/docker.py
> >   "Alex Bennée" <alex.bennee@linaro.org> (maintainer:-----------------...)
> >   Fam Zheng <famz@redhat.com> (maintainer:-----------------...)
> >   "Philippe Mathieu-Daudé" <f4bug@amsat.org> (reviewer:-----------------...)
> >   qemu-devel@nongnu.org (open list:-----------------...)
> > 
> > Fix the issue by inserting a subsystem name line where
> > get_maintainer.pl expects it.
> > 
> > Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> > ---
> >  MAINTAINERS | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index e511ba780f..13c1ac65b7 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -1896,6 +1896,7 @@ F: docs/block-replication.txt
> >  
> >  Build and test automation
> >  -------------------------
> > +Build and test automation
> 
> Would it make sense to use something more specific here? Like "Travis
> and Docker" or so? ... in case we add other subsections in the future?

Unless we are to split off (i.e. more people volunteering maintaining a certain
subset), the list will just go on and on in this line. For not it's already
going to be "Travis, Shippable, Docker and VM test".. so I think a generic
description here is okay, though the duplication is a bit odd (but I don't have
a better idea).

Fam