From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39005) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dfkel-0006JE-4j for qemu-devel@nongnu.org; Thu, 10 Aug 2017 06:25:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dfkeg-0007GT-15 for qemu-devel@nongnu.org; Thu, 10 Aug 2017 06:25:34 -0400 Date: Thu, 10 Aug 2017 12:25:20 +0200 From: Cornelia Huck Message-ID: <20170810122520.48403984.cohuck@redhat.com> In-Reply-To: References: <20170728053610.15770-1-f4bug@amsat.org> <20170728053610.15770-19-f4bug@amsat.org> <87mv7aumav.fsf@linaro.org> <8e2e3aa5-4459-76ba-3309-af6b30220346@amsat.org> <87fud0vmak.fsf@linaro.org> <3e66742f-04c5-51b7-de3a-f692c0637bc5@amsat.org> <20170810104604.342af354.cohuck@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 18/47] MAINTAINERS: add missing TCG entry List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Philippe =?UTF-8?B?TWF0aGlldS1EYXVkw6k=?= , Alex =?UTF-8?B?QmVubsOpZQ==?= , Eric Blake , Fam Zheng , Paolo Bonzini , Richard Henderson , QEMU Trivial , QEMU Developers , Markus Armbruster , "Dr. David Alan Gilbert" On Thu, 10 Aug 2017 10:22:54 +0100 Peter Maydell wrote: > On 10 August 2017 at 09:46, Cornelia Huck wrote: > > The root problem is "some files have no maintainers". The reasons range > > from "forgot to include the file in the pattern" (easily fixed), over > > "file is updated via a script" (the linux-headers case), to "nobody > > feels up to the task" (which is the worst case). > > > > In most cases, I don't think the recent contributors list is very > > helpful. Somebody who simply did a tree-wide rename is unlikely to be > > able to make a good judgment about a complicated logic change. Just > > printing qemu-devel as the address to send this to is probably better; > > unfortunately, it may cause patches to languish on the list if nobody > > takes pity on them. > > > > Do we need someone collecting non-trivial patches like that, who either > > pesters others or picks up the patches themselves? > > The problem is that if nobody's feeling up to the task of taking > on a particular single file which has no maintainer, then it's > definitely true that nobody's going to feel up to taking on > the entire collection of unmaintained files... > > I think the UI (giving no consideration to how we might implement > this!) would ideally be something like: > * if anybody mails a patch which touches an "unmaintained" file, > a robot should send a reply along the lines of "thanks for the > patch; unfortunately file X is not maintained so it may be > tricky to get patch review for this. You'll need to be > persistent and do more of the legwork than if you were patching > a file that did have an active maintainer" so contributors > know when they've wandered into the swamp That's a good idea. > * some mechanism for easily finding patches to unmaintained > files which haven't got review yet, so that anybody with some > spare time and interest can move some of them along (the idea > being to spread the load rather than trying to designate a > particular "owner" for this headache) Can maybe patchew set a special flag for patches that only touch unmaintained files? > * ditto for finding patches to unmaintained files which have got > review but which haven't been committed > > thanks > -- PMM