From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24638 invoked by alias); 9 Nov 2004 05:00:02 -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 24499 invoked from network); 9 Nov 2004 04:59:55 -0000 Received: from unknown (HELO legolas.inter.net.il) (192.114.186.24) by sourceware.org with SMTP; 9 Nov 2004 04:59:55 -0000 Received: from zaretski ([80.230.149.108]) by legolas.inter.net.il (MOS 3.5.3-GR) with ESMTP id DBE44763 (AUTH halo1); Tue, 9 Nov 2004 06:58:04 +0200 (IST) Date: Tue, 09 Nov 2004 05:00:00 -0000 From: "Eli Zaretskii" To: Daniel Jacobowitz Message-ID: <01c4c618$Blat.v2.2.2$0b838560@zahav.net.il> Content-Transfer-Encoding: 7BIT Content-Type: text/plain; charset=ISO-8859-1 CC: cagney@gnu.org, gdb-patches@sources.redhat.com In-reply-to: <20041109011458.GA32113@nevyn.them.org> (message from Daniel Jacobowitz on Mon, 8 Nov 2004 20:14:58 -0500) Subject: Re: [commit] Add add_setshow_enum_cmd, use in mips Reply-to: Eli Zaretskii 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> <20041109011458.GA32113@nevyn.them.org> X-SW-Source: 2004-11/txt/msg00149.txt.bz2 > Date: Mon, 8 Nov 2004 20:14:58 -0500 > From: Daniel Jacobowitz > Cc: cagney@gnu.org, gdb-patches@sources.redhat.com > > 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. I think self-approved patches are only appropriate when the person who submits them is an expert in the area of the patch. > 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 could have some rule that, when there's no specific maintainer for an area, a patch posted as an RFA could go in if unreviewed for some time. > If there is no expert in the area, who will know more about the code > than the global maintainers generally, why ask for review? The fact that there's no appointed area maintainer does not mean that there are no people who know about the related code. It could just mean that no maintainer feels he/she can always be responsive enough to step forward and suggest themselves as area maintainers for that area. > I assume that all of the active maintainers can handle coding > style, and general "is this a gross hack" checks, on their own. Matters of design are always best handled by peer reviews, in my experience. Something that isn't a gross hack can still be not the best design.