[libvirt] [libvirt-python 10/18] virDomainMigrate3 and virDomainMigrateToURI3 not added until 1.1.0

[Eric Blake]

On 11/18/2013 03:44 PM, Doug Goldstein wrote:
> The functions virDomainMigrate3 and virDomainMigrateToURI3 were not
> added to libvirt until v1.1.0.

Is there a method to how you picked which functions to work around?  It
would be nicer to rearrange this series from newest to oldest (1.1.3
[virConnectGetCPUModelNames], 1.1.1 [virDomainCreateWithFiles,
virDomainCreateXMLWithFiles, virDomainSetMemoryStatsPeriod], 1.1.0 ...)
rather than ad hoc.

As to this patch:

> +#if LIBVIR_CHECK_VERSION(1,1,0)

Still the style question.

> @@ -7337,14 +7339,18 @@ static PyMethodDef libvirtMethods[] = {
>      {(char *) "virDomainSendKey", libvirt_virDomainSendKey, METH_VARARGS, NULL},
>      {(char *) "virDomainMigrateGetCompressionCache", libvirt_virDomainMigrateGetCompressionCache, METH_VARARGS, NULL},
>      {(char *) "virDomainMigrateGetMaxSpeed", libvirt_virDomainMigrateGetMaxSpeed, METH_VARARGS, NULL},
> +#if LIBVIR_CHECK_VERSION(1,1,0)
>      {(char *) "virDomainMigrate3", libvirt_virDomainMigrate3, METH_VARARGS, NULL},
>      {(char *) "virDomainMigrateToURI3", libvirt_virDomainMigrateToURI3, METH_VARARGS, NULL},
> +#endif /* LIBVIR_CHECK_VERSION(1,1,0) */

Looks reasonable.

>      {(char *) "virDomainBlockPeek", libvirt_virDomainBlockPeek, METH_VARARGS, NULL},
>      {(char *) "virDomainMemoryPeek", libvirt_virDomainMemoryPeek, METH_VARARGS, NULL},
>      {(char *) "virDomainGetDiskErrors", libvirt_virDomainGetDiskErrors, METH_VARARGS, NULL},
>      {(char *) "virNodeGetMemoryParameters", libvirt_virNodeGetMemoryParameters, METH_VARARGS, NULL},
>      {(char *) "virNodeSetMemoryParameters", libvirt_virNodeSetMemoryParameters, METH_VARARGS, NULL},
> +#if LIBVIR_CHECK_VERSION(1,0,0)
>      {(char *) "virNodeGetCPUMap", libvirt_virNodeGetCPUMap, METH_VARARGS, NULL},
> +#endif /* LIBVIR_CHECK_VERSION(1,0,0) */

Oops, looks like this hunk belongs in 9/18.

-- 
Eric Blake   eblake 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: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20131119/91ee5bea/attachment-0001.sig>