[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