Re: [RFA/testsuite] Remove all remaining gdb_suppress_entire_file

Mark Kettenis
2006-08-08T20:10:56Z

> Date: Tue, 8 Aug 2006 15:00:52 -0400
> From: Daniel Jacobowitz <drow@false.org>
>
> > gdb_suppress_entire_file used to issue a WARNING. There was never
> > supposed to be any ERROR - they were caused by cascading failures,
> > DejaGNU getting confused, and usually the first symptom was a fake
> > "timeout" while trying to load a file into GDB. And that generated an
> > ERROR.
> >
> > After Joel's patch I believe we'll only get an UNTESTED. If you'd
> > rather we could leave the warning?
>
> Hi Mark,
>
> Did you have any opinion on this? One way or another I'd like to get
> this patch squared away.

Hmm, I guess what I really want is an UNTESTED if the failure was to
be expected (for example the ADA tests if there is no ADA compiler on
the system) and an error if something went wrong that shouldn't have
gone wrong (an ICE from GCC on one of the testsuite code snippets).
However, there probably isn't always a clear distinction between the
two. For example, do we expect C++ snippets to compile on all
systems? I suppose Joel's patch is progress; we can always tweak
things later if we feel the UNTESTEDs are inappropriate.

Mark