From: Francois Gouget Subject: [PATCH 2/2] testbot: Prepare support for the "Task: test" status lines. Message-Id: Date: Thu, 18 Oct 2018 11:15:14 +0200 (CEST) In-Reply-To: References: Signed-off-by: Francois Gouget --- 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