[PATCH 2/2] testbot: Prepare support for the "Task: test" status lines.
Francois Gouget
fgouget at codeweavers.com
Thu Oct 18 04:15:14 CDT 2018
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
These will help to more reliably report build issues as such for Wine
VMs.
testbot/lib/WineTestBot/LogUtils.pm | 20 ++++++++++++--------
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/testbot/lib/WineTestBot/LogUtils.pm b/testbot/lib/WineTestBot/LogUtils.pm
index 1f3831bba9..4751f75e53 100644
--- a/testbot/lib/WineTestBot/LogUtils.pm
+++ b/testbot/lib/WineTestBot/LogUtils.pm
@@ -83,7 +83,11 @@ sub ParseTaskLog($)
foreach my $Line (<$LogFile>)
{
chomp $Line;
- if ($Line eq "Task: ok")
+ if ($Line eq "Task: tests")
+ {
+ ; # Ignore it for now
+ }
+ elsif ($Line eq "Task: ok")
{
$Result ||= "ok";
}
@@ -122,6 +126,12 @@ sub GetLogLineCategory($)
{
my ($Line) = @_;
+ if (# Build messages
+ $Line =~ /^\+ \S/ or
+ $Line =~ /^Task: (?:ok|tests)/)
+ {
+ return "info";
+ }
if (# Git errors
$Line =~ /^CONFLICT / or
$Line =~ /^error: patch failed:/ or
@@ -130,7 +140,7 @@ sub GetLogLineCategory($)
$Line =~ /: error: / or
$Line =~ /^make: [*]{3} No rule to make target / or
$Line =~ /^Makefile:[0-9]+: recipe for target .* failed$/ or
- $Line =~ /^Task: (?!ok)/ or
+ $Line =~ /^Task: / or
# Typical perl errors
_IsPerlError($Line))
{
@@ -150,12 +160,6 @@ sub GetLogLineCategory($)
{
return "boterror";
}
- if (# Build messages
- $Line =~ /^\+ \S/ or
- $Line =~ /^Task: ok/)
- {
- return "info";
- }
return "none";
}
--
2.19.1
More information about the wine-devel
mailing list