[PATCH] testbot/WineSendLog: Fix identifying and reporting timeouts to the mailing list.

Francois Gouget fgouget at codeweavers.com
Fri Jul 13 05:07:04 CDT 2018


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---

Fixes the issue identified by Huw.

 testbot/bin/WineSendLog.pl | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/testbot/bin/WineSendLog.pl b/testbot/bin/WineSendLog.pl
index f095bce8d..ac14e11dc 100755
--- a/testbot/bin/WineSendLog.pl
+++ b/testbot/bin/WineSendLog.pl
@@ -139,9 +139,9 @@ sub ReadLog($$$)
       while (! $Found && defined($Line = <LOG>))
       {
         $Line =~ s/[\r\n]*$//;
-        if ($Line =~ m/${BaseName}:${TestSet} done/)
+        if ($Line =~ m/${BaseName}:${TestSet}(?::[0-9a-f]+)? done/)
         {
-          if ($Line =~ m/${BaseName}:${TestSet} done \(258\)/)
+          if ($Line =~ m/${BaseName}:${TestSet}(?::[0-9a-f]+)? done \(258\)/)
           {
             push @Messages, "The test timed out";
           }
@@ -192,7 +192,7 @@ sub CompareLogs($$$$)
       {
         if ($Line =~ m/: Test failed: / || 
             $Line =~ m/: unhandled exception [0-9a-fA-F]{8} at / ||
-            $Line =~ m/Timeout/i)
+            $Line =~ m/The test timed out/)
         {
           $Messages .= "$Line\n";
         }
@@ -301,7 +301,7 @@ EOF
             print SENDMAIL "\n$CurrentDll:\n";
             $PrintedDll = $CurrentDll;
           }
-          if ($Line =~ m/^[^:]+:([^ ]+) done \(258\)/)
+          if ($Line =~ m/^[^:]+:([^ ]+)(?::[0-9a-f]+)? done \(258\)/)
           {
             print SENDMAIL "$1: The test timed out\n";
           }
-- 
2.18.0



More information about the wine-devel mailing list