From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 16907 invoked by alias); 17 Jan 2003 14:26:40 -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 16894 invoked from network); 17 Jan 2003 14:26:37 -0000 Received: from unknown (HELO touchme.toronto.redhat.com) (216.138.202.10) by sources.redhat.com with SMTP; 17 Jan 2003 14:26:37 -0000 Received: from redhat.com (totem.toronto.redhat.com [172.16.14.242]) by touchme.toronto.redhat.com (Postfix) with ESMTP id 751DF800086; Fri, 17 Jan 2003 09:26:37 -0500 (EST) Message-ID: <3E28129D.9050907@redhat.com> Date: Fri, 17 Jan 2003 14:26:00 -0000 From: Fernando Nasser Organization: Red Hat , Inc. - Toronto User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1) Gecko/20020827 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Michael Elizabeth Chastain Cc: ac131313@redhat.com, drow@mvista.com, gdb-patches@sources.redhat.com Subject: Re: [patch/rfc] Remove all setup_xfail's from testsuite/gdb.mi/ References: <200301162006.h0GK65K18945@duracef.shout.net> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2003-01/txt/msg00638.txt.bz2 Michael Elizabeth Chastain wrote:> My two cents ... > > Daniel J suggests that we keep making improvements: > > XFAIL->KFAIL > > random XFAIL->analyzed XFAIL > > XFAIL->PASS > > The problem is that, in the source code, "setup_xfail" looks the same > for both our crap legacy XFAIL's and the nice new analyzed xfail's. > > Perhaps a little mechanism like "gdb_mark_external_fail" would help. > Then "grep xfail" would find only the shrinking pool of old stuff. > You are right, we need some way to distinguish what have been verified and what has not yet been. But if we do what Andrew proposes and get rid of all old PR and HP bug number markings we can make the distinction visually. setup_xfail lines without a bug number have not yet been verified. There aren't that many that would make it so hard, and the poorly marked ones related to stabs or old compilers do show up in your reports very clearly (just diff with a dwarf2 on a new compiler results). And as the reports point out, there are only 71 that are not that case. Just one note: there are cases where the XFAIL is produced by a 'xfail' command inside a '-re' clause. One variation of your idea: The case for debugger format and compiler version is so common that I would feel tempted to have a special function for that. Something like gdb_compiler_related_xfail + could be "stabs", "dwarf2", "any" not sure. A list? An expression? Suggestions? + one or more target specs, as currently (we may even make it optional meaning *-*-*) a bug id, which, of course, will be categorized as "external" > >>This I definitely like. "Cantfix"? > > > I propose "external". > > I find "cantfix" to be a bit arrogant and a bit negative. And it doesn't > distinguish between "I can't fix this because I don't have the resources" > versus "I can't fix this because I can show you that binutils is feeding > gdb incorrect / incomplete information". > Agreed. -- Fernando Nasser Red Hat - Toronto E-Mail: fnasser@redhat.com 2323 Yonge Street, Suite #300 Toronto, Ontario M4P 2C9