On 12/24/2013 08:47 AM, Jean-Baptiste Rouault wrote:
While working on adding virDomain*Stats support to the vbox driver,
we
found bugs in the VirtualBox API C bindings. These bugs have been fixed
in versions 4.2.20 and 4.3.4.
However, the changes in the C bindings are incompatible with the vbox_CAPI_v4_2.h
and vbox_CAPI_v4_3.h files which are bundled in libvirt source code. This is why the
following patch adds vbox_CAPI_v4_2_20.h and vbox_CAPI_v4_3_4.h.
We tried to keep compatibility with older VirtualBox 4.2.x and 4.3.x releases so we
added a "SPECIAL_VERSION" identifier to conditionnaly include the right
header.
I'm not really pleased with this "SPECIAL_VERSION" identifier, maybe we
could instead
increase the precision of "VBOX_API_VERSION", for example 4002 would become
4002000.
This would permit us to select the right header based on the VBOX_API_VERSION only, what
do you think ?
Jean-Baptiste Rouault (1):
vbox: add support for v4.2.20+ and v4.3.4+
src/Makefile.am | 4 +-
src/vbox/vbox_CAPI_v4_2_20.h | 9001 +++++++++++++++++++++++++++++++++++
src/vbox/vbox_CAPI_v4_3_4.h | 10321 +++++++++++++++++++++++++++++++++++++++++
This patch is HUGE (620k, so the moderation queue is currently holding
it as oversized, compared to the normal 150k limit). Is there any way
to break it into smaller pieces, or compress it before sending to the
list, or merely point to an external repo containing the patch, so that
we aren't chewing up lots of bandwidth on mostly mechanical code?
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library
http://libvirt.org