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. Pavel > -- > 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