<div dir="ltr">Sorry for troubling. I'm checking the release note and just find I missed this.<div>This state existed because it is in the middle of something (say I am rewriting the whole network driver and storage driver for vbox.)<div>And I did forget to remove vbox[0-9][0-9]NetworkDriver since we don't need it any more. So this patch just fixed it up.</div><div>What's more, the whole vboxGetDrivers is removed after rewriting the storage driver.</div></div><div>Taowei</div></div><div class="gmail_extra"><br><div class="gmail_quote">2014-10-04 2:57 GMT+08:00 Martin Kletzander <span dir="ltr"><<a href="mailto:mkletzan@redhat.com" target="_blank">mkletzan@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Oct 03, 2014 at 08:50:47PM +0200, Martin Kletzander wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Since 87dea4fcffb2d1b120e7841adc0e3d<u></u>8ea97ed777 vboxGetDrivers() is not<br>
used for getting the vbox network driver.  The only call the code does<br>
is using NULL as the @networkDriver_ret param , but the code still used<br>
vbox[0-9][0-9]NetworkDriver that didn't exist anymore.<br>
<br>
Signed-off-by: Martin Kletzander <<a href="mailto:mkletzan@redhat.com" target="_blank">mkletzan@redhat.com</a>><br>
---<br>
</blockquote>
<br></span>
I didn't do 'format-patch' and 'send-email' doesn't add notes.  So I'm<br>
attaching them now:<br>
<br>
I'm not pushing this as a build-breaker because I sense there might be<br>
more to this.  The thing is that it doesn't fail for anyone else I<br>
talked to, neither does it fail for out test suite.<br>
<br>
I don't quite understand how it could worked (ever), since the only<br>
definition of vbox22NetworkDriver (and others) was extern throughout<br>
the whole git history.  I'm running autogen using --with-vbox even<br>
though I don't have any virtualbox libraries/binaries installed, but<br>
there is no such requirement for compiling the code.  And I don't<br>
think the libraries would export a virNetworkDriver data struct<br>
either.<br>
<br>
Anyway, feel free to let me know where the real problem is and I'll be<br>
glad to help with that either.<br>
<br>--<br>
libvir-list mailing list<br>
<a href="mailto:libvir-list@redhat.com">libvir-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/libvir-list" target="_blank">https://www.redhat.com/mailman/listinfo/libvir-list</a><br></blockquote></div><br></div>