On Fri, Sep 23, 2016 at 09:43:53AM +0200, Pavel Hrdina wrote:
On Fri, Sep 23, 2016 at 08:38:01AM +0200, Martin Kletzander wrote:On Thu, Sep 22, 2016 at 05:08:57PM +0200, Pavel Hrdina wrote: >This patch also removes device data for qemu-1.2.0 as it was removed for >qemu-kvm-1.2.0 by commit ae3e29e6e. They are not required because we >parse only version from help output and return with error that this qemu >is too new to use help parsing. > >Signed-off-by: Pavel Hrdina <phrdina@xxxxxxxxxx> >--- > tests/qemuhelpdata/qemu-1.2.0 | 1 + You should also remove this file ^^ and fix testHelpStrParsing() not to load it if expecting error (and basically not being used).The thing with virQEMUCapsParseHelpStr() is that it parses qemu version from the help output so to make the test meaningful we need to provide output of "qemu -help". This tests the code that reject to parse help if qemu is too new.
Well, we could then provide just that one line if we're cleaning up some stuff. But I don't have a huge preference for that. We'll hopefully remove all the help data 15 years from now when we remove help parsing and support for old QEMU versions =)
Pavel-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
Attachment:
signature.asc
Description: Digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list