[tools] testbot/LogUtils: Don't modify ParseWineTestReport()'s timeout lines.
Francois Gouget
fgouget at codeweavers.com
Fri Feb 21 04:37:05 CST 2020
Changing the timeout lines for reporting prevented the parser from
recognizing them and lead to extraneous missing failure / todo / skip
errors.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
testbot/lib/WineTestBot/LogUtils.pm | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 863cca6bf4..1a87f174b7 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -532,17 +532,18 @@ sub ParseWineTestReport($$$)
if (GetReportLineCategory($Line) eq "error")
{
# Make the timeout messages more user-friendly
- if ($Line =~ m/^[^:]+:([^:]*):[0-9a-f]+ done \(258\)/)
+ my $ErrLine = $Line;
+ if ($ErrLine =~ /^[^:]+:([^:]*):[0-9a-f]+ done \(258\)/)
{
my $Unit = $1;
- $Line = $Unit ne "" ? "$Unit: Timeout" : "Timeout";
+ $ErrLine = $Unit ne "" ? "$Unit: Timeout" : "Timeout";
}
if (!$CurGroup)
{
$CurGroup = _AddLogGroup($LogInfo, $CurGroupName, $CurGroupLineNo || $LineNo);
}
- _AddLogError($LogInfo, $CurGroup, $Line, $LineNo);
+ _AddLogError($LogInfo, $CurGroup, $ErrLine, $LineNo);
}
if ($Line =~ m%^([_.a-z0-9-]+):([_a-z0-9]*) (start|skipped) (?:-|[/_.a-z0-9-]+) (?:-|[.0-9a-f]+)\r?$%)
--
2.20.1
More information about the wine-devel
mailing list