Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Daniel Jacobowitz <drow@mvista.com>
To: George France <france@handhelds.org>
Cc: gdb@sources.redhat.com, clp@iol.unh.edu
Subject: Re: exit status of 'make check'
Date: Tue, 13 Nov 2001 08:47:00 -0000	[thread overview]
Message-ID: <20011125124718.A25769@nevyn.them.org> (raw)
In-Reply-To: <01112512374200.05740@shadowfax.middleearth>

On Sun, Nov 25, 2001 at 12:37:42PM -0500, George France wrote:
> > That pretty much means that it did.  You can grep for ^FAIL: to see how
> > many there are.  I'm hoping to see the GDB testsuite pass entirely for
> > some target, someday...
> 
> Greetings,
> 
> Even if all the testsuites have a FAILed status, 'make check' should not 
> return a non-zero exit code unless there is a failure in the 'make check' 
> programs. The programs that comprise 'make check' should succeed.  
> 
> For example, if the "gdb.base/annota1.exp: breakpoint info" testsuite fails, 
> it is a failure of that testsuite, not 'make check'.  Not finding 'runtest' 
> would be an example of a failure in 'make check', that should return a 
> non-zero exit code.
> 
> Currently 'make check' always returns a non-zero exit code indicating the 
> testsuite results are unreliable due to a failure in 'make check'. 
> 
> I hope this makes things clearer.

I really can't agree.  I don't care (in an idealized world in which
we've finished fixing the GDB testsuite, which we're working on...)
what went wrong.  I'd consider make check to have failed if a test
failed - for use in, say, unattended builds.

From runtest's man page:

EXIT CODES
       runtest sets the exit code to 1 if any of the tests failed, or
       sets it to 0 if all the
       tests passed.

DejaGNU seems to agree with me on this one.

-- 
Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer


  reply	other threads:[~2001-11-25 17:47 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-11-11 10:33 Michael Elizabeth Chastain
2001-11-11 11:30 ` George France
2001-11-11 22:14   ` Daniel Jacobowitz
2001-11-12 15:02     ` George France
2001-11-13  8:47       ` Daniel Jacobowitz [this message]
2001-11-13  9:53         ` George France
2001-11-13 10:05           ` Daniel Jacobowitz
2001-11-13 10:26             ` George France
2001-11-25 20:59               ` George France
  -- strict thread matches above, loose matches on Subject: below --
2001-11-13 10:34 Michael Elizabeth Chastain
2001-11-13 12:38 ` George France
2001-11-25 22:21   ` George France
2001-11-25 21:11 ` Michael Elizabeth Chastain
2001-11-10 10:17 George France

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=20011125124718.A25769@nevyn.them.org \
    --to=drow@mvista.com \
    --cc=clp@iol.unh.edu \
    --cc=france@handhelds.org \
    --cc=gdb@sources.redhat.com \
    /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