On Thursday 30 April 2009 14:26:06 Daniel P. Berrange wrote:
> On Wed, Apr 29, 2009 at 01:14:05PM +0200, Pritesh Kothari wrote:
> > Hi All,
> >
> > PATCH 1/1: contains changes in version detection and moving a global
> > variable in the global struct, so that no more global vairables exists.
> >
> > + if (uVersion >= 2001052 && uVersion < 2002051) {
> > + DEBUG0("VirtualBox API version: 2.2");
> > + driver = &vbox22Driver;
> > + } else if (uVersion >= 2002051 && uVersion < 2005051) {
> > + DEBUG0("VirtualBox API version: 2.5");
> > + driver = &vbox25Driver;
> > + } else {
> > + DEBUG0("Unsupport VirtualBox API version");
> > + }
>
> This bit seems to cause a link error for the driver, because nothing
> is defined the vbox25Driver symbol. I guess you have another patch
> pending to add vbox_V2_5.c ?
>
> I just #if 0'd the 'else if' block to make it link
Thanks for #if0'ing it, sorry forgot it.
yes i have a patch for 2.5 but much of work is still underway due to the
recent added functionality in VirtualBox, post it soon once it is stable
enough.
Regards,
Pritesh
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list