Francois Gouget : testbot: Better check for the "No build or test VM" disposition.
Alexandre Julliard
julliard at winehq.org
Tue Sep 25 12:29:48 CDT 2018
Module: tools
Branch: master
Commit: d70d002ded822c29a63a8f97ec62aa9fd43199cd
URL: https://source.winehq.org/git/tools.git/?a=commit;h=d70d002ded822c29a63a8f97ec62aa9fd43199cd
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Tue Sep 25 16:58:15 2018 +0200
testbot: Better check for the "No build or test VM" disposition.
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>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
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 469fedc..96e4a0d 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!");
}
More information about the wine-cvs
mailing list