On 5/6/19 2:42 PM, Andrea Bolognani wrote:
On Fri, 2019-05-03 at 10:47 +0200, Boris Fiuczynski wrote: [...]+++ b/tests/domaincapstest.c @@ -453,6 +453,10 @@ mymain(void) DO_TEST_QEMU("4.0.0", "caps_4.0.0", "/usr/bin/qemu-system-x86_64", NULL, "x86_64", VIR_DOMAIN_VIRT_KVM); + + DO_TEST_QEMU("4.0.0", "caps_4.0.0", + "/usr/bin/qemu-system-s390x", NULL, + "s390x", VIR_DOMAIN_VIRT_KVM);It's pretty annoying that we have to update this test program manually every time new .replies are added. Easy to miss, too, and in fact we seem to only have coverage for about half the possible input files. We should modify this so that dropping .replies into the source directory is enough for the test program to pick them up, just like qemucapabilitiestest and qemucaps2xmltest do. But that's work for another day, so in the meantime Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>
Andrea, thanks for the reviewing, reworking and pushing. -- Mit freundlichen Grüßen/Kind regards Boris Fiuczynski IBM Deutschland Research & Development GmbH Vorsitzender des Aufsichtsrats: Matthias Hartmann Geschäftsführung: Dirk Wittkopp Sitz der Gesellschaft: Böblingen Registergericht: Amtsgericht Stuttgart, HRB 243294 -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
