[libvirt] [PATCH v3 1.5/11] admin: introduce virAdmGetVersion

Erik Skultety eskultet at redhat.com
Mon Nov 30 09:47:54 UTC 2015


>> +/**
>> + * virAdmGetVersion:
>> + * @libVer: where to store the library version
>> + *
>> + * Provides version information. @libVer is the version of the
>> library and will
>> + * allways be set unless an error occurs in which case an error code
>> and a
>> + * generic message will be returned. @libVer format is as follows:
>> + * major * 1,000,000 + minor * 1,000 + release.
>> + *
>> + * NOTE: To get the remote side library version use
>> virAdmDaemonGetLibVersion
> 
> Either you meant virAdmConnectGetLibVersion here or
> virAdmDaemonGetVersion but not the one you used.  You should probably
> keep the *Connect* version for now as the future of my patch for
> renaming was not yet decided upon.  If it is, I'll change it in the
> rename patch itself.
> 
> ACK to [012].5

Fixed the minor issue and pushed the series, thank you for all reviews.

Erik




More information about the libvir-list mailing list