Francois Gouget : testbot: Better check for the "No build or test VM" disposition.
Module: tools Branch: master Commit: d70d002ded822c29a63a8f97ec62aa9fd43199cd URL: https://source.winehq.org/git/tools.git/?a=commit;h=d70d002ded822c29a63a8f97... Author: Francois Gouget <fgouget(a)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(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)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!"); }
participants (1)
-
Alexandre Julliard