[PATCH] testbot: Better check for the "No build or test VM" disposition.

Francois Gouget fgouget at codeweavers.com
Tue Sep 25 09:58:15 CDT 2018


Use a direct test instead of an indirect one.
Also the $Impacts->{UnitCount} field has been renamed.

Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 testbot/lib/WineTestBot/Patches.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/testbot/lib/WineTestBot/Patches.pm b/testbot/lib/WineTestBot/Patches.pm
index 469fedcdc..96e4a0dda 100644
--- a/testbot/lib/WineTestBot/Patches.pm
+++ b/testbot/lib/WineTestBot/Patches.pm
@@ -266,7 +266,7 @@ sub Submit($$$)
   if ($NewJob->Steps->IsEmpty())
   {
     # This may be a Wine patch but there is no suitable VM to test it!
-    if ($Impacts->{UnitCount})
+    if ($BuildVMs->IsEmpty() and $WineVMs->IsEmpty())
     {
       $self->Disposition("No build or test VM!");
     }
-- 
2.19.0




More information about the wine-devel mailing list