On 09/15/2017 07:07 PM, Eduardo Habkost wrote: > Cleber and I are volunteering to review and queue patches for the > Python scripts and modules in scripts/. > > I'm setting "M: Odd fixes" because not all scripts are actively > maintained. > > Signed-off-by: Eduardo Habkost <ehabk...@redhat.com> > --- > Even before this patch is merged, I plan to send a pull request > including some patches for the Python code soon. > --- > MAINTAINERS | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 2c333aba21..2c3b8ecde7 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -1390,6 +1390,13 @@ S: Maintained > F: include/sysemu/cryptodev*.h > F: backends/cryptodev*.c > > +Python scripts > +M: Eduardo Habkost <ehabk...@redhat.com> > +M: Cleber Rosa <cr...@redhat.com> > +S: Odd fixes > +F: scripts/qmp/* > +F: scripts/*.py > + > QAPI > M: Markus Armbruster <arm...@redhat.com> > M: Michael Roth <mdr...@linux.vnet.ibm.com> >
I rather like the thought of having dedicated Python maintainers who can review python code with an eye for what is idiomatic. I am fairly certain the python I write is functional, but I'm rarely sure it's what a python programmer would do. I suppose you are intentionally omitting any python that exists as part of the test infrastructure, however? John
