From b8f36d3eb8d04ec15a53816d78c851868916054f Mon Sep 17 00:00:00 2001
From: Jelmer Vernooij <jelmer@samba.org>
Date: Fri, 12 Jun 2009 00:19:02 +0200
Subject: selftest/buildfarm: Don't assume that a reason being specified is an
 indication of failure.

---
 selftest/output/buildfarm.pm | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/selftest/output/buildfarm.pm b/selftest/output/buildfarm.pm
index 09b84de99e..8612cfbc76 100644
--- a/selftest/output/buildfarm.pm
+++ b/selftest/output/buildfarm.pm
@@ -87,9 +87,16 @@ sub control_msg($$)
 	}
 }
 
-sub end_testsuite($$$$$$)
+sub end_testsuite($$$$$)
 {
-	my ($self, $name, $result, $unexpected, $reason) = @_;
+	my ($self, $name, $result, $reason) = @_;
+
+	my $unexpected;
+	if ($result eq "failure" or $result eq "fail" or $result eq "error") {
+		$unexpected = 1;
+	} else {
+		$unexpected = 0;
+	}
 
 	BuildFarm::end_testsuite($name, ($self->{last_time} - $self->{START_TIME}), 
 		                     (not $unexpected), $self->{test_output}->{$name}, 
-- 
cgit