[libvirt] [PATCH 5/8] qemuhelptest: regenerate data for qemu-1.2.0

Pavel Hrdina phrdina at redhat.com
Fri Sep 23 07:43:53 UTC 2016


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 at redhat.com>
> >---
> > 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 at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160923/58cc91e1/attachment-0001.sig>


More information about the libvir-list mailing list