From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 30180 invoked by alias); 9 Nov 2004 01:15:08 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 29790 invoked from network); 9 Nov 2004 01:14:58 -0000 Received: from unknown (HELO nevyn.them.org) (66.93.172.17) by sourceware.org with SMTP; 9 Nov 2004 01:14:58 -0000 Received: from drow by nevyn.them.org with local (Exim 4.34 #1 (Debian)) id 1CRKb4-0008O7-3J; Mon, 08 Nov 2004 20:14:58 -0500 Date: Tue, 09 Nov 2004 01:15:00 -0000 From: Daniel Jacobowitz To: Eli Zaretskii Cc: cagney@gnu.org, gdb-patches@sources.redhat.com Subject: Re: [commit] Add add_setshow_enum_cmd, use in mips Message-ID: <20041109011458.GA32113@nevyn.them.org> Mail-Followup-To: Eli Zaretskii , cagney@gnu.org, gdb-patches@sources.redhat.com References: <4183BD22.3090905@gnu.org> <01c4bed6$Blat.v2.2.2$fa231b20@zahav.net.il> <41856ECA.2060701@gnu.org> <01c4bfcd$Blat.v2.2.2$299ef260@zahav.net.il> <20041101051257.GA11134@nevyn.them.org> <01c4c057$Blat.v2.2.2$4cacd760@zahav.net.il> <20041101223716.GB28889@nevyn.them.org> <01c4c096$Blat.v2.2.2$d4f57520@zahav.net.il> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <01c4c096$Blat.v2.2.2$d4f57520@zahav.net.il> User-Agent: Mutt/1.5.5.1+cvs20040105i X-SW-Source: 2004-11/txt/msg00144.txt.bz2 [Eli, I apologize for the delay in responding; I didn't mean to drop this conversation on the floor, but I haven't had much time for GDB lately.] On Tue, Nov 02, 2004 at 06:44:48AM +0200, Eli Zaretskii wrote: > > Date: Mon, 1 Nov 2004 17:37:16 -0500 > > From: Daniel Jacobowitz > > Cc: cagney@gnu.org, gdb-patches@sources.redhat.com > > > > I only see a point for maintainers to post RFAs when (A) they can > > not approve the patch themselves or (B) they are not > > confident/happy/sure with the approach. > > I'm astonished: you are, in effect, saying that the patch review > process exists only because some meaningless bureaucratic rule does > not permit a single person to do whatever he/she wants. I kinda > thought that the patch review is the default, except when the patch > comes from an expert whom we trust to be good enough not to need that. I'm certainly not trying to say that! But self-review seems like a reasonable practice to me and my experience reading gdb-patches shows it to be a pretty common one. Most patches are self-approved. Some maintainers tend to post patches to areas without specific maintainers as RFA's; others don't. The other thing experience tells me is that patches posted as an RFA, by someone who could self-approve it, only very rarely get reviewed. Often they sit for ages. > > We don't operate on consensus > > I thought we should. If not, I don't see much sense in the machinery > that we have in place. To me, the reason for our procedures is to > produce quality code, not just to make an impression of due process. Again, I feel disconnected. I agree with your premises but not your conclusions. If there is no expert in the area, who will know more about the code than the global maintainers generally, why ask for review? I assume that all of the active maintainers can handle coding style, and general "is this a gross hack" checks, on their own. -- Daniel Jacobowitz