Re: [PATCH v2 1/1] MAINTAINERS: Add Python library stanza

[Eduardo Habkost]

On Wed, Sep 23, 2020 at 11:17:01AM -0400, John Snow wrote:
> On 9/23/20 11:14 AM, Markus Armbruster wrote:
> > John Snow <jsnow@redhat.com> writes:
> > 
> > > I'm proposing that I split the actual Python library off from the other
> > > miscellaneous python scripts we have and declare it maintained. Add
> > > myself as a maintainer of this folder, along with Cleber.
> > > 
> > > v2: change python/* to python/, thanks Alex.
> > > 
> > > Signed-off-by: John Snow <jsnow@redhat.com>
> > > Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> > > Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> > > Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
> > > Acked-by: Cleber Rosa <crosa@redhat.com>
> > > Acked-by: Eduardo Habkost <ehabkost@redhat.com>
> > > ---
> > >   MAINTAINERS | 9 ++++++++-
> > >   1 file changed, 8 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 3d17cad19a..c0222ee645 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -2355,11 +2355,18 @@ S: Maintained
> > >   F: include/sysemu/cryptodev*.h
> > >   F: backends/cryptodev*.c
> > > +Python library
> > > +M: John Snow <jsnow@redhat.com>
> > > +M: Cleber Rosa <crosa@redhat.com>
> > > +R: Eduardo Habkost <ehabkost@redhat.com>
> > > +S: Maintained
> > > +F: python/
> > > +T: git https://gitlab.com/jsnow/qemu.git python
> > > +
> > >   Python scripts
> > >   M: Eduardo Habkost <ehabkost@redhat.com>
> > >   M: Cleber Rosa <crosa@redhat.com>
> > >   S: Odd fixes
> > > -F: python/qemu/*py
> > >   F: scripts/*.py
> > >   F: tests/*.py
> > 
> > Separate sections just so you can have the appropriate S:, or is there a
> > deeper logic behind the split?
> > 
> 
> Yes, different intended levels of support. I don't currently have the
> bandwidth to promise support for the miscellaneous Python scripts in
> ./scripts, but I do promise to support to a higher level the code in
> ./python.
> 
> Over time, I intend to migrate things into ./python, but there are some
> series that need to happen first before I start doing that.

Also, the actual owners for most scripts in ./scripts are already
listed elsewhere.  I believe we shouldn't really have M: lines
for the scripts/*.py section, just R:.

-- 
Eduardo