sg-report-flight: Do not report many non-regressions as allowable

In bd648aea2ebe
 sg-report-flight: report allowable regressions separately in summary
whether the regression was allowable was put in $allow, but this
was erroneously not used.

In 4a210cda9cc8
 sg-report-flight: fix allowable failure logic not to reuse $allow for two purposes (!)
that use of the variable, which was never used, was removed.

However, what ought to happen, is that $failv->{Allow} would be set to
1 if the entry matched an allow pattern, and only if it was a blocker.
That makes the logic in "report ... separately" correct.

Signed-off-by: Ian Jackson <ian.jackson@eu.citrix.com>

diff --git a/sg-report-flight b/sg-report-flight
index 5b8f67a3e73325ed33f49f1fffaa1820066e4417..5e362b91d19c757c2e9dd476d1b8f64bac7ce538 100755 (executable)
--- a/sg-report-flight
+++ b/sg-report-flight
@@ -888,6 +888,7 @@ END
             print DEBUG " allow $item?";
             foreach my $allowpat (@allows) {
                 next unless $item =~ m/$allowpat/;
+                $failv->{Allow}= 1;
                 $blocker='';
                 print DEBUG " allowed";
                 last;
@@ -895,7 +896,6 @@ END
             print DEBUG "\n";
         }
        $failv->{Blocker}= $blocker;
-        $failv->{Allow}= 1;
        $failv->{SummaryCore}= $rtups[0][2];
        $failv->{SummaryRefFlight}= $rtups[0][3];
        $failv->{Summary}= $failv->{SummaryCore};