Francois Gouget : testbot: Fix a check for reverting VMs on the Engine startup.

Alexandre Julliard julliard at winehq.org
Wed Oct 11 07:24:43 CDT 2017


Module: tools
Branch: master
Commit: fd5d3d4f40f2b5ceb103832ea25b2bc0e0c90076
URL:    http://source.winehq.org/git/tools.git/?a=commit;h=fd5d3d4f40f2b5ceb103832ea25b2bc0e0c90076

Author: Francois Gouget <fgouget at codeweavers.com>
Date:   Tue Oct 10 23:56:26 2017 +0200

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

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>
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 34d29f6..ee496e2 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";




More information about the wine-cvs mailing list