Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Fernando Nasser <fnasser@redhat.com>
To: Michael Elizabeth Chastain <mec@shout.net>
Cc: ac131313@cygnus.com, drow@mvista.com,
	gdb-patches@sources.redhat.com, Rob Savoye <rob@welcomehome.org>
Subject: RFC: KFAILs [Was: [RFA/mi-testsuite] XFAIL mi*-console.exp]
Date: Fri, 05 Apr 2002 07:58:00 -0000	[thread overview]
Message-ID: <3CADC91A.21753C54@redhat.com> (raw)
In-Reply-To: <200204051523.g35FNGh02714@duracef.shout.net>

Michael Elizabeth Chastain wrote:
> 
> What's the status of KFAIL?
> 

I have it here in my sandbox, but I will have to clean it up a
little bit this Sunday. I have a few questions and comments:


1) All current Dejagnu messages have the bug identification at
the end (between parenthesis).  I want to keep things consistent.
Is that OK?

2) Also, to keep things consistent in Dejagnu, the bug identification
for the kfail would be printed as "(PRMS xxxx)" as it is done
for XFAIL etc., to distinguish from bug_id which is printed as
"(BUG xxxx)".  Is that a problem?

So, a KFAIL would be printed as:

KFAIL: could not run to marker1 (PRMS gdb/999)

Would that make the scripts happy?


3) For KFAILs, the bug identification is mandatory.  In this case
I would like to change the syntax a little bit and have it as the
first argument and issue an error if it is not there.  I would 
still keep the current Dejagnu convention that bug identifications
are strings without a '-' in it.  I am doing this just to be able
to detect that someone forgot to specify it.  I know that there are
other syntax possibilities (like command switches), but I am trying
to be consistent with what is already there.

So, here is an example of a kfail use:

setup_kfail "gdb/999" *-*-*


4) Note that, when a test that was expected to fail due to a known
bug suddenly starts to pass, it becomes a KPASS (as XFAILs do).
It is then time to go and check if the bug was fixed and someone
forgot to remove the setup_kfail or if something changed in the
environment and the bug is not applicable anymore (or the test
is not catching it anymore).  More or less as we should do with
XPASSes that appear instead of XFAILs.  Also, sometimes the target
specification is a little bit too general (the problem does not
happen in all the specified conditions).


5) As soon as we have KFAILs, we will have to re-examine the FAILs
we have, register a bug for what is a gdb bug.  Also, we will be able
to have zero FAILs, so in any platform a non-zero result will mean that
we broke something.  We will also have to re-check everything that was
marked as XFAIL and see if they are actually known GDB bugs.



6) I would like to write a script that, as part of the release, goes 
through the KFAILs, access the Gnats database with the Bug Id and 
fills the BUGS part of the man page (can also generate a report and
put it in a file -- or append it to a Release Notes").  
We can also have a script to compare the new known bugs report with
the previous release one and generate a list of "Bugs fixed in this
release" (this can help people decide if they should upgrade, or stop
people to post just to hear: "I suggest you upgrade").

I will do it in Perl (I still don't know how to programmatically access
the Gnats database though).  But I have very little spare time, so I
will
not mind if someone that can do it sooner volunteers to do this.
 


-- 
Fernando Nasser
Red Hat Canada Ltd.                     E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9


  reply	other threads:[~2002-04-05 15:58 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-04-05  7:23 [RFA/mi-testsuite] XFAIL mi*-console.exp Michael Elizabeth Chastain
2002-04-05  7:58 ` Fernando Nasser [this message]
2002-04-05  9:53 RFC: KFAILs [Was: [RFA/mi-testsuite] XFAIL mi*-console.exp] Michael Elizabeth Chastain
2002-04-05 10:22 ` Daniel Jacobowitz
2002-04-05 13:45   ` Andrew Cagney
2002-04-05 13:51     ` Daniel Jacobowitz
2002-04-05 10:23 ` Fernando Nasser
2002-04-05 11:09 Michael Elizabeth Chastain
2002-04-05 12:07 ` Fernando Nasser
2002-04-07 16:30 ` Rob Savoye
2002-04-08  8:41   ` Fernando Nasser
2002-04-05 11:14 Michael Elizabeth Chastain
2002-04-07 16:35 ` Rob Savoye
2002-04-07 16:53 Michael Elizabeth Chastain
2002-04-07 17:02 ` Rob Savoye
2002-04-07 17:31 Michael Elizabeth Chastain
2002-04-07 17:38 ` Rob Savoye
2002-04-08  8:59 Michael Elizabeth Chastain
2002-04-08  9:06 ` Fernando Nasser

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3CADC91A.21753C54@redhat.com \
    --to=fnasser@redhat.com \
    --cc=ac131313@cygnus.com \
    --cc=drow@mvista.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=mec@shout.net \
    --cc=rob@welcomehome.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox