coreutils
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tests: add extra protection against unexpected exits


From: Bernhard Voelker
Subject: Re: [PATCH] tests: add extra protection against unexpected exits
Date: Wed, 14 Jan 2015 17:40:15 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0

On 01/14/2015 04:48 PM, Jim Meyering wrote:
On Wed, Jan 14, 2015 at 1:24 AM, Bernhard Voelker
<address@hidden> wrote:
On 01/14/2015 02:51 AM, Pádraig Brady wrote:
-compare /dev/null err && fail=1
+test -s err || fail=1

nice one!

Actually, I prefer the former.
When some program produces unexpected output, that use of
compare-vs-/dev/null will ensure that the surprising output is
printed in the test's output. If you use only the "test -s err",
you would have to instrument and rerun in order to see
the offending output.

oops, you're right.

There were a few others, too.
Fixed with the attached.

Have a nice day,
Berny

Attachment: 0001-tests-use-compare-vs-dev-null-instead-of-test-s.patch
Description: Text Data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]