On Wed, May 30, 2018 at 05:03:34PM -0300, Philippe Mathieu-Daudé wrote: > In order to allow out-of-tree acceptance tests with Avocado, > create a symlink in the build tree. >
"In other to allow" seems to imply that running acceptance tests on an out-of-tree build is impossible without this patch, but: [...] > Before: > > build_dir$ avocado run > /full/path/to/sources/qemu/tests/acceptance/boot_linux_console.py > > After > > build_dir$ avocado run tests/acceptance/boot_linux_console.py > This seems to imply that running them is possible, but this patch makes it more convenient. Is this just for convenience, or necessary to allow acceptance tests to run properly? Can this be clarified in the commit message? > I first wanted to try something like: > > echo "[datadir.paths]" > .avocado.conf > echo "test_dir = $source_path/tests/acceptance" >> .avocado.conf > > to run: > > build_dir$ avocado run -t console > > but this doesn't work this way, I'd have to use > > build_dir$ avocado --config .avocado.conf boot_linux_console.py -t console > > which isn't a win. > --- > configure | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/configure b/configure > index a6a4616c3e..0763ebe5ba 100755 > --- a/configure > +++ b/configure > @@ -7213,9 +7213,11 @@ for rom in seabios vgabios ; do > done > > # set up tests data directory > -if [ ! -e tests/data ]; then > - symlink "$source_path/tests/data" tests/data > -fi > +for tests_subdir in acceptance data; do > + if [ ! -e tests/$tests_subdir ]; then > + symlink "$source_path/tests/$tests_subdir" tests/$tests_subdir > + fi > +done > > # set up qemu-iotests in this build directory > iotests_common_env="tests/qemu-iotests/common.env" > -- > 2.17.1 > -- Eduardo
