All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] MAINTAINERS leaves too many files uncovered
Date: Wed, 31 Aug 2016 10:28:37 +0200	[thread overview]
Message-ID: <87wpix4a16.fsf@dusky.pond.sub.org> (raw)
In-Reply-To: <920b7a0b-d031-8c6a-22db-55a5dd04a668@redhat.com> (Paolo Bonzini's message of "Tue, 30 Aug 2016 17:33:08 +0200")

I really shouldn't reply to this, because more maintainer duties is
about the last thing I need, but here goes anyway:

Paolo Bonzini <pbonzini@redhat.com> writes:

> On 30/08/2016 17:29, Markus Armbruster wrote:
>> > > > Active subsystems lacking a maintainer include tilegx, qdev, replay.
>> > > 
>> > > You, Eduardo, me?
>> >
>> > FWIW I was referring to qdev.
>>
>> Jointly?
>
> A subset not including me would also be fine.  But since the git
> fallback still exists, and the code is _obviously_ maintained, I don't
> think there is a particular hurry to add new people to the list.
> Maintainers are most useful when there are a lot of simple patches, not
> when each patch's design must be discussed in the community.

Even then, having someone "own" the discussion is useful: make sure it
happens, and stays on track.

Moreover, not *all* qdev patches need wider discussion.  You're doing a
good job picking up such patches to "unmaintained" code.  Still, having
less "unmaintained" code can only make that job easier.

Last but not least,

    $ scripts/get_maintainer.pl -f hw/core/qdev.c 
    get_maintainer.pl: No maintainers found, printing recent contributors.
    get_maintainer.pl: Do not blindly cc: them on patches!  Use common sense.

    Markus Armbruster <armbru@redhat.com> (commit_signer:4/12=33%)
    David Gibson <david@gibson.dropbear.id.au> (commit_signer:4/12=33%)
    Eric Blake <eblake@redhat.com> (commit_signer:3/12=25%)
    Igor Mammedov <imammedo@redhat.com> (commit_signer:2/12=17%)
    "Andreas Färber" <afaerber@suse.de> (commit_signer:2/12=17%)
    qemu-devel@nongnu.org (open list:All patches CC here)

must look scary for contributors who don't know that qdev "is
_obviously_ maintained".

  reply	other threads:[~2016-08-31  8:28 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-10-20  9:19 [Qemu-devel] MAINTAINERS leaves too many files uncovered Markus Armbruster
2014-10-20 18:36 ` Paolo Bonzini
2014-10-22  8:23 ` Markus Armbruster
2014-10-22  9:00   ` Paolo Bonzini
2014-10-22 11:45     ` Markus Armbruster
2014-10-22 11:50       ` Paolo Bonzini
2014-10-22 13:06         ` Markus Armbruster
2014-10-22 13:22           ` Alexander Graf
2014-10-22 13:36           ` Paolo Bonzini
2014-10-22 11:58       ` [Qemu-devel] MAINTAINERS: please add myself as usb-serial.c and baum.c maintainer Samuel Thibault
2014-10-22 12:51         ` Paolo Bonzini
2014-10-22 13:00         ` Markus Armbruster
2014-10-22 13:53           ` Samuel Thibault
2014-10-22 15:21             ` Markus Armbruster
2014-10-22 12:06       ` [Qemu-devel] MAINTAINERS leaves too many files uncovered Michael Roth
2014-10-22 12:54       ` Luiz Capitulino
2015-09-22  9:13   ` Markus Armbruster
2015-09-22 10:06     ` Daniel P. Berrange
2015-09-22 10:18       ` Paolo Bonzini
2015-09-22 13:42     ` Thomas Huth
2015-09-22 14:58       ` Gerd Hoffmann
2015-09-22 15:13       ` Juan Quintela
2015-09-22 14:58     ` Daniel P. Berrange
2015-09-22 15:40       ` Peter Maydell
2015-09-22 18:07     ` Eric Blake
2015-09-22 19:50       ` Paolo Bonzini
2015-09-23  8:24       ` Kevin Wolf
2015-09-23  8:30         ` Alberto Garcia
2015-09-24 12:49           ` Markus Armbruster
2015-09-24 12:47       ` Markus Armbruster
2015-09-22 18:37     ` John Snow
2016-08-30 11:31     ` Markus Armbruster
2016-08-30 12:26       ` Cornelia Huck
2016-08-30 12:40         ` Paolo Bonzini
2016-08-30 13:29           ` Cornelia Huck
2016-08-30 12:48       ` Paolo Bonzini
2016-08-30 12:57         ` Peter Maydell
2016-08-30 18:31           ` John Snow
2016-08-30 14:47         ` Paolo Bonzini
2016-08-30 15:29           ` Markus Armbruster
2016-08-30 15:33             ` Paolo Bonzini
2016-08-31  8:28               ` Markus Armbruster [this message]
2016-08-30 13:39       ` Cornelia Huck
2016-09-12 13:56       ` Thomas Huth
2016-09-23 12:56       ` Thomas Huth
2016-09-23 14:47         ` Thomas Huth
2018-11-21 18:14       ` Markus Armbruster
2018-11-22 12:22         ` Thomas Huth
2018-11-22 21:56           ` BALATON Zoltan
2018-11-23  5:17             ` Philippe Mathieu-Daudé
2018-11-23 10:56               ` BALATON Zoltan
2018-11-23 11:03                 ` Philippe Mathieu-Daudé
2018-11-26 13:54           ` Markus Armbruster
2014-10-22 14:02 ` Alex Bennée
2014-10-22 15:26   ` Markus Armbruster

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87wpix4a16.fsf@dusky.pond.sub.org \
    --to=armbru@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.