[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry
From: |
mdroth |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] MAINTAINERS: split Monitor (QMP/HMP) entry |
Date: |
Mon, 3 Jun 2013 15:05:03 -0500 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Jun 03, 2013 at 03:54:48PM -0400, Luiz Capitulino wrote:
> On Mon, 03 Jun 2013 21:13:30 +0200
> Paolo Bonzini <address@hidden> wrote:
>
> > Il 03/06/2013 19:17, Luiz Capitulino ha scritto:
> > > This entry doesn't reflect reality for a few years now. This commit
> > > splits it into Human Monitor (HMP), QAPI and QMP. Markus is dropped
> > > as a maintainer.
> > >
> > > This is what we have been for the last few years. Also, it's going
> > > to help me to offload some of this work to someone else in the near
> > > future.
> > >
> > > Signed-off-by: Luiz Capitulino <address@hidden>
> > > ---
> > > MAINTAINERS | 17 +++++++++++++++--
> > > 1 file changed, 15 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 432185a..45f2e68 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -685,11 +685,12 @@ M: Anthony Liguori <address@hidden>
> > > S: Supported
> > > F: vl.c
> > >
> > > -Monitor (QMP/HMP)
> > > +Human Monitor (HMP)
> > > M: Luiz Capitulino <address@hidden>
> > > -M: Markus Armbruster <address@hidden>
> > > S: Supported
> > > F: monitor.c
> > > +F: hmp.c
> > > +F: hmp-commands.hx
> > >
> > > Network device layer
> > > M: Anthony Liguori <address@hidden>
> > > @@ -706,6 +707,11 @@ F: nbd.*
> > > F: qemu-nbd.c
> > > T: git git://github.com/bonzini/qemu.git nbd-next
> > >
> > > +QAPI
> > > +M: Luiz Capitulino <address@hidden>
> >
> > Mike Roth too, perhaps?
>
> I'd like to, but last time I asked he sort of refused :)
>
> Michael, can I add you?
>
Haha, well, I think the discussion back then was whether I should send
QAPI pulls, and I was of the mindset that we should keep sending those
through the QMP tree since most QAPI changes were in support of QMP.
According to the MAINTAINERS documentation though, M: is meant to be
"a set of names that developers can consult when they have a question
about a particular subset and also to provide a set of names to be
CC'd when submitting a patch to obtain appropriate review.", so if
that's the case I'd be happy to add my name and help in that regard.
Re: [Qemu-devel] [PATCH 0/2] MAINTAINERS: QMP: entry update and new maintainers, Eric Blake, 2013/06/03
Re: [Qemu-devel] [PATCH 0/2] MAINTAINERS: QMP: entry update and new maintainers, Anthony Liguori, 2013/06/03