[tools] testbot: Empty configuration regexps match any configuration.

Francois Gouget fgouget at codeweavers.com
Fri Jun 17 09:50:03 CDT 2022


Unlike the failure regexps which must not be empty.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/lib/WineTestBot/LogUtils.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index e25ea3ae27..4c643b9e09 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -1387,7 +1387,7 @@ sub MatchLogFailures($$)
   {
     # Ignore failures that don't apply to this configuration
     my $ConfigRegExp = $Failure->ConfigRegExp;
-    my $Match = eval { $ConfigRegExp and $ConfigName =~ /$ConfigRegExp/ };
+    my $Match = eval { !$ConfigRegExp or $ConfigName =~ /$ConfigRegExp/ };
     next if (!$Match);
 
     my $UnitFailures = $FailureTree{$Failure->ErrorGroup}->{$Failure->TestUnit} ||= [];
-- 
2.30.2




More information about the wine-devel mailing list