Re: [PATCH v2 3/6] tests: qemuxml2xml: Convert virtio-*transitional to TEST_CAPS

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/16/19 4:58 AM, Andrea Bolognani wrote:
> On Mon, 2019-04-15 at 19:09 -0400, Cole Robinson wrote:
> [...]
>> +    DO_TEST_CAPS_LATEST("virtio-transitional");
>> +    DO_TEST_CAPS_LATEST("virtio-non-transitional");
> 
> We should also add
> 
>   DO_TEST_CAPS_VER("virtio-transitional", "3.1.0");
>   DO_TEST_CAPS_VER("virtio-non-transitional", "3.1.0");
> 
> same as xml2argv. You can do that in a separate patch, though.
> 
> Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>
> 

Hmm I'm not sure if those bits will actually test any different code
paths for xml2xml? I'd check coverage output but lcov is broken with
gcc9 on f30

Anyways thanks for the reviews, I've pushed these now

Thanks,
Cole

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux