From: Ian Jackson <iwj@woking.cam.xci-test.com>
Date: Tue, 12 Feb 2013 19:07:17 +0000 (+0000)
Subject: sg-report-flight: do not write to MRO for recursive flights, so we avoid writing... 
X-Git-Tag: wip.xsm-13-actual.base~476^2~90
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=a7a731205bf49828eb8c70ca8a6964a86f8b082f;p=people%2Fiwj%2Fosstest.git

sg-report-flight: do not write to MRO for recursive flights, so we avoid writing "tolerable" when it's not
---

diff --git a/sg-report-flight b/sg-report-flight
index cdf30762..82903ada 100755
--- a/sg-report-flight
+++ b/sg-report-flight
@@ -713,19 +713,21 @@ END
               scalar(@failures), "\n";
     }
 
-    if (!@failures) {
-        print MRO "tolerable\nperfect\n" or die $!;
-         $fi->{Overall}.= "Perfect :-)\n";
-    } elsif (grep { $_->{Blocker} eq 'regression' } @failures) {
-        $fi->{OutcomeSummary}= "regressions - $fi->{OutcomeSummary}";
-        $fi->{Overall}.= "Regressions :-(\n";
-    } elsif (!grep { $_->{Blocker} } @failures) {
-        $fi->{OutcomeSummary}= "tolerable $fi->{OutcomeSummary}";
-        print MRO "tolerable\n" or die $!
-            unless defined $heisen_why;
-        $fi->{Overall}.= "Failures :-/ but no regressions.\n";
-    } else {
-        $fi->{Overall}.= "Failures and problems with tests :-(\n";
+    if (!$heisen_why) {
+	if (!@failures) {
+	    print MRO "tolerable\nperfect\n" or die $!;
+	    $fi->{Overall}.= "Perfect :-)\n";
+	} elsif (grep { $_->{Blocker} eq 'regression' } @failures) {
+	    $fi->{OutcomeSummary}= "regressions - $fi->{OutcomeSummary}";
+	    $fi->{Overall}.= "Regressions :-(\n";
+	} elsif (!grep { $_->{Blocker} } @failures) {
+	    $fi->{OutcomeSummary}= "tolerable $fi->{OutcomeSummary}";
+	    print MRO "tolerable\n" or die $!
+		unless defined $heisen_why;
+	    $fi->{Overall}.= "Failures :-/ but no regressions.\n";
+	} else {
+	    $fi->{Overall}.= "Failures and problems with tests :-(\n";
+	}
     }
 
     return @failures;