From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 9215 invoked by alias); 1 Aug 2008 17:40:54 -0000 Received: (qmail 9205 invoked by uid 22791); 1 Aug 2008 17:40:53 -0000 X-Spam-Check-By: sourceware.org Received: from sibelius.xs4all.nl (HELO sibelius.xs4all.nl) (82.92.89.47) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 01 Aug 2008 17:40:24 +0000 Received: from brahms.sibelius.xs4all.nl (kettenis@localhost.sibelius.xs4all.nl [127.0.0.1]) by brahms.sibelius.xs4all.nl (8.14.3/8.14.3) with ESMTP id m71HbR7I025837; Fri, 1 Aug 2008 19:37:27 +0200 (CEST) Received: (from kettenis@localhost) by brahms.sibelius.xs4all.nl (8.14.3/8.14.3/Submit) id m71HbRm3029431; Fri, 1 Aug 2008 19:37:27 +0200 (CEST) Date: Fri, 01 Aug 2008 17:40:00 -0000 Message-Id: <200808011737.m71HbRm3029431@brahms.sibelius.xs4all.nl> From: Mark Kettenis To: tromey@redhat.com CC: stan@codesourcery.com, gdb-patches@sources.redhat.com In-reply-to: (message from Tom Tromey on Fri, 01 Aug 2008 10:20:18 -0600) Subject: Re: RFA: Remove gdb-events References: <487975AF.4030306@codesourcery.com> Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2008-08/txt/msg00011.txt.bz2 > X-Spam-Check-By: sourceware.org > Cc: gdb-patches@sources.redhat.com > From: Tom Tromey > Reply-To: Tom Tromey > Date: Fri, 01 Aug 2008 10:20:18 -0600 > Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm > Sender: gdb-patches-owner@sourceware.org > X-XS4ALL-DNSBL-Checked: mxdrop110.xs4all.nl checked 209.132.176.174 against DNS blacklists > X-CNFS-Analysis: v=1.0 c=1 a=881O8E-6XdsA:10 a=2XsNsC2eOfWF1qdpl05llQ==:17 > a=qo2X1CjQWrMaJWK6xZoA:9 a=9A5clqTUE07V6g9w-5tw_btH0rQA:4 > a=jEp0ucaQiEUA:10 a=WcGY3-TIXCwA:10 > X-Virus-Scanned: by XS4ALL Virus Scanner > X-XS4ALL-Spam-Score: 0.0 () none > X-XS4ALL-Spam: NO > Envelope-To: m.m.kettenis@xs4all.nl > X-UIDL: 1217607658._smtp.mxdrop110.31168,S=3733 > > >>>>> "Tom" == Tom Tromey writes: > > Tom> I'd like to propose changing this for gdb. I don't think merging is > Tom> even needed. Folks can always look in subdir ChangeLog.old... > > Any comment on this? > > The concrete proposal is: > > * Rename gdb/doc/ChangeLog -> gdb/doc/ChangeLog.old > * Rename gdb/testsuite/ChangeLog -> gdb/testsuite/ChangeLog.old > > ... and all future patches in these areas will just use gdb/ChangeLog. > > The rationale is that many patches span these directories, so keeping > their ChangeLog entries together makes sense. On the flip side, > keeping them separate does not seem to add much value -- it is just > more work when preparing a patch. While committing a testcase for bugs you fix is great, there is no need to commit the testcase simultaniously with the fix. It is better to actually commit the testcase as soon as the problem is known and not wait until you have a fix. Personally I like the fact that the testsuite Changelog is seperate. It means I don't get distracted by testsuite changes when I'm looking for code changes. And testsuite changes, which occur relatively infrequent, don't get burried between lots of code changes.