From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15393 invoked by alias); 16 Dec 2002 18:22:42 -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 15375 invoked from network); 16 Dec 2002 18:22:40 -0000 Received: from unknown (HELO crack.them.org) (65.125.64.184) by sources.redhat.com with SMTP; 16 Dec 2002 18:22:40 -0000 Received: from nevyn.them.org ([66.93.61.169] ident=mail) by crack.them.org with asmtp (Exim 3.12 #1 (Debian)) id 18O1lX-0004HY-00; Mon, 16 Dec 2002 14:23:03 -0600 Received: from drow by nevyn.them.org with local (Exim 3.36 #1 (Debian)) id 18Nzu1-0005tr-00; Mon, 16 Dec 2002 13:23:41 -0500 Date: Mon, 16 Dec 2002 10:34:00 -0000 From: Daniel Jacobowitz To: Elena Zannoni Cc: "J. Johnston" , gdb-patches@sources.redhat.com Subject: Re: Recent MI patches broke the testsuite Message-ID: <20021216182341.GA22508@nevyn.them.org> Mail-Followup-To: Elena Zannoni , "J. Johnston" , gdb-patches@sources.redhat.com References: <20021215211132.GA31984@nevyn.them.org> <3DFE15BA.4080207@redhat.com> <15870.6748.994104.390618@localhost.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <15870.6748.994104.390618@localhost.redhat.com> User-Agent: Mutt/1.5.1i X-SW-Source: 2002-12/txt/msg00491.txt.bz2 On Mon, Dec 16, 2002 at 01:24:28PM -0500, Elena Zannoni wrote: > J. Johnston writes: > > My bad. :( The new mi-basics.exp testcase is my usual sanity test > > for this change. It was updated and in fact works fine as it is expecting > > the new behavior. > > > > I did not look at the correct gdb.log for the full mi testsuite. > > As you have noted, the lib/mi-support.exp routine mi_reinitialize_dir > > is using -environment-directory and is expecting the old output. > > > > I have a patch for lib/mi=support.exp to handle the new syntax which I have > > included. Since I require approval for patches I will revert the > > original patch and resubmit with the added change unless Elena says > > otherwise. > > > > If this patch fixes the testsuite, no need to revert the other one, > just check this in. Don't forget the ChangeLog. > > Maybe Daniel can verify that it fixes his problem? Yes, it does, except for the mi-basics.exp part. And that's minor. -- Daniel Jacobowitz MontaVista Software Debian GNU/Linux Developer