[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