On Fri, Sep 26, 2025 at 10:34:01AM +0200, Thomas Huth wrote: > On 26/09/2025 07.15, Gustavo Romero wrote: > > Add check-venv target as a dependency for the functional tests. This > > causes Python modules listed in pythondeps.toml, under the testdeps > > group, to be installed when 'make check-functional' is executed to > > prepare and run the functional tests. > > > > Signed-off-by: Gustavo Romero <[email protected]> > > Suggested-by: Thomas Huth <[email protected]> > > --- > > tests/Makefile.include | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/tests/Makefile.include b/tests/Makefile.include > > index 3538c0c740..d012a9b25d 100644 > > --- a/tests/Makefile.include > > +++ b/tests/Makefile.include > > @@ -109,7 +109,7 @@ $(FUNCTIONAL_TARGETS): > > @$(MAKE) SPEED=thorough $(subst -functional,-func,$@) > > .PHONY: check-functional > > -check-functional: > > +check-functional: check-venv > > I just noticed that there's still a problem: If you run "make > check-functional-aarch64" immediately after configuring + compiling QEMU in > a fresh folder for the first time, the functional tests fail with: > > ModuleNotFoundError: No module named 'pygdbmi' > > We either need to add dependencies to the check-functional-<arch> targets, > too, or we have to make sure that tests still get properly skipped in the > case that pygdbmi has not been installed into the venv yet.
We already have a decorator for skipping tests when modules are missing, so we should add usage of that. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
