]> xenbits.xensource.com Git - people/liuw/osstest.git/commitdiff
Executive: Do update flights_harness_touched for play flights
authorIan Jackson <ian.jackson@eu.citrix.com>
Fri, 1 Jul 2016 17:28:44 +0000 (18:28 +0100)
committerIan Jackson <Ian.Jackson@eu.citrix.com>
Mon, 5 Sep 2016 14:03:19 +0000 (15:03 +0100)
Have \bplay\b simply bypass the blessing check, but not the harness
revision update which follows.

Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Osstest/JobDB/Executive.pm

index c0af21c457d91b94e6af3944bf832fa428ee7613..51c1ebbd3e7c6d836852a863567cdc99ca5a523c 100644 (file)
@@ -108,9 +108,11 @@ END
 
     die "modifying flight $fl but flight not found\n"
         unless defined $bless;
-    return if $bless =~ m/\bplay\b/;
-    die "modifying flight $fl blessing $bless expected @$flok\n"
-        unless grep { $_ eq $bless } @$flok;
+
+    unless ($bless =~ m/\bplay\b/) {
+       die "modifying flight $fl blessing $bless expected @$flok\n"
+           unless grep { $_ eq $bless } @$flok;
+    }
 
     my $rev = get_harness_rev();