Francois Gouget : testbot: Only send an email when a job is really done.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: 891d048b168fcd04b2f5f2984bf65873028d56c0
URL:    https://source.winehq.org/git/tools.git/?a=commit;h=891d048b168fcd04b2f5f2984bf65873028d56c0

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Fri Aug 10 20:11:20 2018 +0200

testbot: Only send an email when a job is really done.

In particular don't send a report if it is requeued or was canceled.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 testbot/bin/Engine.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testbot/bin/Engine.pl b/testbot/bin/Engine.pl
index 464b6db..4478d47 100755
--- a/testbot/bin/Engine.pl
+++ b/testbot/bin/Engine.pl
@@ -307,7 +307,7 @@ sub HandleJobStatusChange($$$)
     LogMsg "Invalid JobKey $JobKey in jobstatuschange message\n";
   }
 
-  if ($OldStatus eq "running" && $NewStatus ne "running")
+  if ($OldStatus eq "running" and $NewStatus !~ /^(?:canceled|queued|running)$/)
   {
     my $Pid = fork;
     if (!defined $Pid)