[tools] patches: Take the .testbot file Status into account.

Francois Gouget fgouget at codeweavers.com
Thu Jun 16 10:20:39 CDT 2022


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
So far whether a patch passed the tests or not seems to only be based on 
the wine-devel emails, despite the information being available in the 
locale .testbot file.
---
 patches/update | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/patches/update b/patches/update
index 5d20f2a11..494d2e157 100755
--- a/patches/update
+++ b/patches/update
@@ -210,7 +210,8 @@ foreach my $file (readdir DIR)
             {
                 utf8::decode($_);
                 last if /^$/;
-                if (/^Job-ID: (.*)$/) { $patch{"testjob"} = $1; }
+                if (/^Status: (.*)$/) { $patch{"testbot"} = $1; }
+                elsif (/^Job-ID: (.*)$/) { $patch{"testjob"} = $1; }
                 elsif (/^URL: (.*)$/) { $patch{"testurl"} = $1; }
             }
             close LOG;
@@ -254,7 +255,6 @@ sub assign_parent($)
         if ($patch->{"from_marvin"})
         {
             $parent->{"testbot"} = "<a href=\"data/$patch->{id}\">Failed</a>";
-            $parent->{"testclass"} = "testbot botfail";
         }
         return $parent;
     }
@@ -320,6 +320,10 @@ foreach my $file (sort { $patches{$b}->{"order"} <=> $patches{$a}->{"order"} } k
     {
         printf INDEX "<td class=\"failmark\">X</td>";
     }
+    if ($patch->{"testbot"} and $patch->{"testbot"} ne "OK")
+    {
+        $patch->{"testclass"} .= " botfail";
+    }
     printf INDEX "<td class=\"%s\">%s</td>", $patch->{"testclass"}, $patch->{"testbot"};
     if ($patch->{"testjob"} && $patch->{"testurl"})
     {
-- 
2.30.2



More information about the wine-devel mailing list