[Tools] testbot: Fix a check for reverting VMs on the Engine startup.

Francois Gouget fgouget at codeweavers.com
Tue Oct 10 16:56:26 CDT 2017


If the Engine was starting up while a VM was reverting, it would power
off that VM instead of letting the revert run its course.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 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 34d29f62..ee496e2a 100755
--- a/testbot/bin/Engine.pl
+++ b/testbot/bin/Engine.pl
@@ -195,7 +195,7 @@ sub Cleanup(;$$)
         next;
       }
       elsif (($VM->Status eq "reverting" or $VM->Status eq "sleeping") and
-             defined $VM->ChildPid and kill(0, $VM->ChildPid))
+             defined $VM->ChildPid and !kill(0, $VM->ChildPid))
       {
         # This VM is still being reverted. Let that process run its course.
         LogMsg "$VMKey is being reverted\n";
-- 
2.14.2




More information about the wine-patches mailing list