From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <lvm2-cvs-return-1765-listarch-lvm2-cvs=sources.redhat.com@sourceware.org>
Received: (qmail 25320 invoked by alias); 16 Feb 2009 16:49:22 -0000
Received: (qmail 25306 invoked by uid 9699); 16 Feb 2009 16:49:22 -0000
Date: Mon, 16 Feb 2009 16:49:00 -0000
Message-ID: <20090216164922.25304.qmail@sourceware.org>
From: mornfall@sourceware.org
To: lvm-devel@redhat.com, lvm2-cvs@sourceware.org
Subject: LVM2/test harness.c
Mailing-List: contact lvm2-cvs-help@sourceware.org; run by ezmlm
Precedence: bulk
List-Id: <lvm2-cvs.sourceware.org>
List-Subscribe: <mailto:lvm2-cvs-subscribe@sourceware.org>
List-Post: <mailto:lvm2-cvs@sourceware.org>
List-Help: <mailto:lvm2-cvs-help@sourceware.org>, <http://sourceware.org/lists.html#faqs>
Sender: lvm2-cvs-owner@sourceware.org
X-SW-Source: 2009-02/txt/msg00013.txt.bz2

CVSROOT:	/cvs/lvm2
Module name:	LVM2
Changes by:	mornfall@sourceware.org	2009-02-16 16:49:22

Modified files:
	test           : harness.c 

Log message:
	Only fail when tests have failed, but not when they have been just skipped.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/LVM2/test/harness.c.diff?cvsroot=lvm2&r1=1.1&r2=1.2

--- LVM2/test/harness.c	2009/02/12 19:54:45	1.1
+++ LVM2/test/harness.c	2009/02/16 16:49:21	1.2
@@ -153,7 +153,7 @@
 	       npassed + nfailed + nskipped, npassed, nfailed, nskipped);
 
 	/* print out a summary */
-	if (nfailed || nskipped ) {
+	if (nfailed || nskipped) {
 		for (i = 1; i < argc; ++ i) {
 			switch (status[i]) {
 			case FAILED:
@@ -165,7 +165,7 @@
 			}
 		}
 		printf("\n");
-		return 1;
+		return nfailed > 0 || die;
 	}
 	return !die;
 }