From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18183 invoked by alias); 10 Nov 2004 23:41:54 -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 18144 invoked from network); 10 Nov 2004 23:41:49 -0000 Received: from unknown (HELO nevyn.them.org) (66.93.172.17) by sourceware.org with SMTP; 10 Nov 2004 23:41:49 -0000 Received: from drow by nevyn.them.org with local (Exim 4.34 #1 (Debian)) id 1CS25w-00005R-6u; Wed, 10 Nov 2004 18:41:44 -0500 Date: Wed, 10 Nov 2004 23:41:00 -0000 From: Daniel Jacobowitz To: Eli Zaretskii Cc: Mark Kettenis , cagney@gnu.org, gdb-patches@sources.redhat.com Subject: Re: [commit] Add add_setshow_enum_cmd, use in mips Message-ID: <20041110234143.GA32661@nevyn.them.org> Mail-Followup-To: Eli Zaretskii , Mark Kettenis , cagney@gnu.org, gdb-patches@sources.redhat.com References: <20041101223716.GB28889@nevyn.them.org> <01c4c096$Blat.v2.2.2$d4f57520@zahav.net.il> <20041109011458.GA32113@nevyn.them.org> <01c4c618$Blat.v2.2.2$0b838560@zahav.net.il> <4190E1F8.7000203@gnu.org> <20041109184221.GB13359@nevyn.them.org> <01c4c6dd$Blat.v2.2.2$a3bf1ea0@zahav.net.il> <01c4c766$Blat.v2.2.2$ce1fd200@zahav.net.il> <200411102142.iAALgEPM095582@elgar.sibelius.xs4all.nl> <01c4c77c$Blat.v2.2.2$a7e52160@zahav.net.il> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <01c4c77c$Blat.v2.2.2$a7e52160@zahav.net.il> User-Agent: Mutt/1.5.5.1+cvs20040105i X-SW-Source: 2004-11/txt/msg00220.txt.bz2 On Thu, Nov 11, 2004 at 01:25:05AM +0200, Eli Zaretskii wrote: > > Date: Wed, 10 Nov 2004 22:42:14 +0100 (CET) > > From: Mark Kettenis > > CC: cagney@gnu.org, drow@false.org, gdb-patches@sources.redhat.com > > > > Sorry, but I have to disagree here. For the (unfortunately) limited > > number of people that contribute several patches in a week this is a > > significant problem. When I'm working on a particular area I often > > find myself making multiple changes to the same file. If I have to > > post a patch and wait a week before I can check it in, I have two > > options: > > > > 1. Juggle with the patches for a week, risking an accidental commit of > > stuff belonging to a different patch to the same file, or dropping > > a patch completely in the process. > > > > 2. Postpone further work on that part of GDB until the week is over > > and the patch has been committed. > > > > Neither option is good for GDB. I completely agree with Mark. > That might be tough, but we all do precisely that when the file in > question is not in our maintainership area. You've said that quite a few times now :-) But this isn't a documented policy, and it doesn't seem to be a widely followed one, either. This seems to be the heart of our disagreement. > Yes, it slows down the development a bit, but I don't think the patch > rate is our most important goal. The rate is important, but code > quality and clean design are IMHO more important. We can't improve the code quality and design, which have suffered over time, if we can't make progress on patches. Indeed, the longer it takes for every single change to go in, the less incentive there is to clean anything up! I'm sure losing incentive to fix things. -- Daniel Jacobowitz