From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45245) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1emiZk-0001MP-2Q for qemu-devel@nongnu.org; Fri, 16 Feb 2018 11:09:28 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1emiZg-0000Km-TU for qemu-devel@nongnu.org; Fri, 16 Feb 2018 11:09:28 -0500 Date: Fri, 16 Feb 2018 17:09:04 +0100 From: Cornelia Huck Message-ID: <20180216170904.5933125f.cohuck@redhat.com> In-Reply-To: <4a71adf2-f617-74c2-cf7e-34517ac6aade@redhat.com> References: <20180216121810.56ed961e.cohuck@redhat.com> <8b721785-5581-8936-2517-1fa00db3dabd@redhat.com> <4a71adf2-f617-74c2-cf7e-34517ac6aade@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [Process] QEMU submaintainers and pull requests List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Peter Maydell , qemu-s390x , QEMU Developers On Fri, 16 Feb 2018 16:08:39 +0100 Paolo Bonzini wrote: > On 16/02/2018 16:03, Peter Maydell wrote: > > On 16 February 2018 at 14:53, Paolo Bonzini wrote: > >> On 16/02/2018 12:25, Peter Maydell wrote: > >>> On 16 February 2018 at 11:18, Cornelia Huck wrote: > >>> The block folks are already doing this, so we should just > >>> formalize what they're doing at the moment I guess. From my point > >>> of view as long as there's something I can easily filter > >>> in/out in the email body or subject so I don't get confused > >>> (and which doesn't require me to update my filters every time > >>> a new subsystem switches to using submaintainer pulls!) > >>> I don't mind about the rest of it. Maybe a subject line with > >>> 'PULL SUBSYSTEM s390x' (ditto block, etc etc) ? > >> > >> What about just requiring you to be in the "To" or "Cc" fields? That > >> works pretty well for Linux. > > > > It would require existing submaintainers to change their process, > > though -- currently not all pull requests are to/cc me. > > Yeah, but that would not be too hard. There are only about 40 > maintainers, and if anybody misses the news they would notice fairly > quickly. :) Well, I wouldn't mind (I send pull requests to Peter anyway, and I'd certainly want s390x pull requests addressed to me as well). We could use both (subject prefix and To:/Cc:) for a belt-and-suspenders approach. > > BTW, Fam and I are also planning to improve Patchew so that it is better > at detecting pull requests and especially merged pull requests. Cool. Would a certain formatting be helpful for patchew as well? Or do you plan to parse the cover letter to hopefully find the output of git request-pull and get the correct base from that?