Am 09.08.2017 um 23:55 hat Paolo Bonzini geschrieben: > These are never used by "check", with one exception that does not need > $QEMU_OPTIONS. Keep them in common.rc, which will be soon included only > by the tests. > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com>
> diff --git a/tests/qemu-iotests/common b/tests/qemu-iotests/common > index 50720f080f..f58e56fc40 100644 > --- a/tests/qemu-iotests/common > +++ b/tests/qemu-iotests/common > @@ -454,11 +454,3 @@ fi > # > list=`sort $tmp.list` > rm -f $tmp.list $tmp.tmp $tmp.sed > - > -[ "$QEMU" = "" ] && _fatal "qemu not found" > -[ "$QEMU_IMG" = "" ] && _fatal "qemu-img not found" > -[ "$QEMU_IO" = "" ] && _fatal "qemu-io not found" > - > -if [ "$IMGPROTO" = "nbd" ] ; then > - [ "$QEMU_NBD" = "" ] && _fatal "qemu-nbd not found" > -fi Hm, does this mean that instead of ./check failing when a binary is missing, we try each test case now and each one fails with the same error message? *tries it out* Okay, it's already broken today because the strings are never empty but contain the name of the wrapper functions, but it's still bad behaviour. Instead of just telling me that the binary is missing like it used to work, I get tons of test case diffs. Kevin
