public inbox for ltp@lists.linux.it
 help / color / mirror / Atom feed
From: Mike Frysinger <vapier@gentoo.org>
To: ltp-list@lists.sourceforge.net
Subject: Re: [LTP] [PATCH] ppoll01: add signal() syscall to register the signal handler
Date: Sat, 26 Sep 2009 07:25:28 -0400	[thread overview]
Message-ID: <200909260725.29279.vapier@gentoo.org> (raw)
In-Reply-To: <200909241229.32866.chandru@in.ibm.com>


[-- Attachment #1.1: Type: Text/Plain, Size: 737 bytes --]

On Thursday 24 September 2009 02:59:32 Chandru wrote:
>  3. Takes away
>  printing the type of error ( i.e strerror(errno) ) since it only prints
>  the last error seen and does not report the failures seen in the cases
>  before the last failed one. The reporting of NG or OK shoudl take care of
>  letting the user know the cases that have passed and cases that have
>  failed.
>
> -                 	   	tst_resm(TFAIL, "%s failed - errno = %d : %s", TCID,
>  TEST_ERRNO, strerror(TEST_ERRNO));
>  +                 	   	tst_resm(TFAIL, "%s failed", TCID);

why dont you *fix* the issues you see with this output instead of making the 
output even more useless ?  also, this should be converted to TFAIL|TTERRNO.
-mike

[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 401 bytes --]

------------------------------------------------------------------------------
Come build with us! The BlackBerry&reg; Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9&#45;12, 2009. Register now&#33;
http://p.sf.net/sfu/devconf

[-- Attachment #3: Type: text/plain, Size: 155 bytes --]

_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list

  parent reply	other threads:[~2009-09-26 11:25 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-24  6:59 [LTP] [PATCH] ppoll01: add signal() syscall to register the signal handler Chandru
2009-09-24 20:28 ` K.D. Lucas
2009-09-25  5:24   ` Garrett Cooper
2009-09-26 11:25 ` Mike Frysinger [this message]
2009-10-01  7:27   ` Chandru
2009-10-01  8:16     ` Chandru
2009-10-01 16:39     ` Mike Frysinger

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=200909260725.29279.vapier@gentoo.org \
    --to=vapier@gentoo.org \
    --cc=ltp-list@lists.sourceforge.net \
    /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