From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 15627 invoked by alias); 22 May 2003 17:40:15 -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 14694 invoked from network); 22 May 2003 17:39:53 -0000 Received: from unknown (HELO localhost.redhat.com) (207.219.125.131) by sources.redhat.com with SMTP; 22 May 2003 17:39:53 -0000 Received: from redhat.com (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id 4A6542B5E; Thu, 22 May 2003 13:39:39 -0400 (EDT) Message-ID: <3ECD0B5B.9060900@redhat.com> Date: Thu, 22 May 2003 17:40:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.2) Gecko/20030223 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Corinna Vinschen Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA]: File-I/O patch, testsuite References: <20021121101107.V24928@cygbert.vinschen.de> <3ECC02D9.3060306@redhat.com> <20030522065211.GH19367@cygbert.vinschen.de> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-05/txt/msg00426.txt.bz2 > On Wed, May 21, 2003 at 06:51:05PM -0400, Andrew Cagney wrote: > >> >However, this tests are currently running always, regardless if the >> >target implements this functionality or not. I'm not quite sure >> >about the best way to skip these tests for those targets... > >> >> I was going to suggest: >> > >> ># test only on a remote target board >> >if {! [is_remote target]} { >> > return >> >} > > > Yeah, we could add this. Sounds about right. but I think: if [target_info exists gdb,noinferiorio] { verbose "Skipping interrupt.exp because of noinferiorio." return } is more correct. It's the lack of the I/O feature, and not the remoteness that determines if the test should run. >> but it doesn't hurt to have this run native (should work?). > > > It doesn't make any sense, though. :-) It would make maintenance easier. The more mainstream the test, the more likely it is to be maintained. CF remote.exp which gets little attention (suffering constant bit rot) because it is only run on remote targets :-( >> I don't understand why: >> >> +catch "system \"chmod -f +w dir2.fileio.test\"" >> +catch "system \"rm -rf *.fileio.test\"" >> >> appears twice. > > > Paranoid mode. The catches removes all files produced by the fileio tests > of a previous testsuite run and it does so at the end to not let them > hanging around when the test finishes. The test itself needs to have > a clean start, otherwise it produces a couple of annoying FAILs. Ah, ok. Andrew