From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13610 invoked by alias); 1 Aug 2005 01:57:59 -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 13020 invoked by uid 22791); 1 Aug 2005 01:57:55 -0000 Received: from nevyn.them.org (HELO nevyn.them.org) (66.93.172.17) by sourceware.org (qpsmtpd/0.30-dev) with ESMTP; Mon, 01 Aug 2005 01:57:55 +0000 Received: from drow by nevyn.them.org with local (Exim 4.52) id 1DzPYw-0004az-4g; Sun, 31 Jul 2005 21:57:54 -0400 Date: Mon, 01 Aug 2005 01:57:00 -0000 From: Daniel Jacobowitz To: Nick Roberts Cc: gdb-patches@sources.redhat.com Subject: Re: PATCH: tests for MI commands Message-ID: <20050801015754.GG30901@nevyn.them.org> Mail-Followup-To: Nick Roberts , gdb-patches@sources.redhat.com References: <17117.55351.716393.763492@farnswood.snap.net.nz> <20050724211946.GB798@nevyn.them.org> <17126.51233.887175.402254@farnswood.snap.net.nz> <20050727002721.GA27261@nevyn.them.org> <17126.62115.642102.742744@farnswood.snap.net.nz> <20050727035503.GA30670@nevyn.them.org> <17129.56052.792623.200657@farnswood.snap.net.nz> <20050731212522.GB30901@nevyn.them.org> <17133.23564.530245.592625@farnswood.snap.net.nz> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <17133.23564.530245.592625@farnswood.snap.net.nz> User-Agent: Mutt/1.5.8i X-SW-Source: 2005-08/txt/msg00003.txt.bz2 On Mon, Aug 01, 2005 at 11:17:32AM +1200, Nick Roberts wrote: > > > I would rather copy my changes to mi-stack.exp and mi-var-child.exp over to > > > mi2-stack.exp and mi2-var-child.exp. Creating new files mi-var-cmd.c, > > > mi-basics.c etc seems more complicated. Is this an acceptable short term > > > solution? > > > > It's not complicated at all. Please see the attached patch, which I've > > checked in. It would be even simpler than this if folks had used > > ${srcfile} to begin with. > > Well now some mi-*.exp files e.g mi-var-block.exp use var-cmd.c while > others, namely mi-var-child.exp, use mi-var-child.c. Clearly, though if > you do the work its simpler (for me!). Every other test in the testsuite (well, not true. All the recent ones anyway!) has its own source file. We've found this to be the best option - because really, most of the time when you change a source file in the testsuite, it's so that you can add or improve tests, not fix bugs in the programs being tested. So duplication is less problematic than the lockstep tangle that we encountered here. So that's the general principle. > > > Tested on i686-pc-linux-gnu, where it resolves 18 FAILs and assorted > > ERRORs. > > Thanks. I'll test them on my other machine, but I'm sure they will work. I > guess the idea is that mi2-*.exp tests shouldn't change although I'm still not > convinced that they are useful - perhaps we can discuss that later. Yes, precisely. I'm still a bit undecided on the issue of the mi2-*.exp tests, but I think that either (A) they shouldn't change or (B) they should be removed. For now the course of inertia is (A). > In future, I'll run the whole testsuite before submitting patches, to save you > and others hassle. Thank you! -- Daniel Jacobowitz CodeSourcery, LLC