[libvirt] patch for openvz driver to support vzctl 3.1

Eric Blake eblake at redhat.com
Wed Apr 4 20:37:16 UTC 2012


On 04/04/2012 10:59 AM, Ilja Livenson wrote:
> Hi,
> 
> there was a recent update of vzctl from openvz, which made openvz
> driver crash: https://www.redhat.com/archives/libvirt-users/2012-March/msg00203.html
> Not sure about the correct procedure, but the patch is very simple and
> is shown here: https://bugzilla.redhat.com/show_bug.cgi?id=809895 .
> 
> Basically, openvz dropped strict version numbering (3.1 vs 3.1.0),
> which caused parsing to fail.

Thanks for the heads up.  Some advice for next time: when posting a
patch to a list, post the actual patch, and not a URL to some other
location where the patch is embedded.  It goes back to the principle of
making the maintainer's life as easy as possible - if you make the
maintainer do the legwork that you could have done, then your patch goes
to the bottom of the queue.  Also, your patch was reversed (it is
traditional to diff old as -, new as +).

That said, I've given you enough grief for one day, so ACK, and I pushed
your patch as well as adding you to AUTHORS.  Let me know if you prefer
an alternate spelling.

> patch is quite simple:
>
> --- src/openvz/openvz_conf.c
> +++ src/openvz/openvz_conf.c
> @@ -99,7 +99,7 @@ openvzExtractVersionInfo(const char *cmd
>      if ((tmp = STRSKIP(tmp, "vzctl version ")) == NULL)
>          goto cleanup;
>
> -    if (virParseVersionString(tmp, &version, true) < 0)
> +    if (virParseVersionString(tmp, &version, false) < 0)
>          goto cleanup;
>
>      if (retversion)

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120404/4bc33f82/attachment-0001.sig>


More information about the libvir-list mailing list