qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


From: Fam Zheng
Subject: Re: [Qemu-devel] [PATCH] MAINTAINERS: Fix subsystem name for "Build and test automation"
Date: Fri, 22 Sep 2017 11:30:26 +0800
User-agent: Mutt/1.8.3 (2017-05-23)

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" <address@hidden> (maintainer:-----------------...)
> >   Fam Zheng <address@hidden> (maintainer:-----------------...)
> >   "Philippe Mathieu-Daudé" <address@hidden> (reviewer:-----------------...)
> >   address@hidden (open list:-----------------...)
> > 
> > Fix the issue by inserting a subsystem name line where
> > get_maintainer.pl expects it.
> > 
> > Signed-off-by: Eduardo Habkost <address@hidden>
> > ---
> >  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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]