Re: [libvirt] [PATCH] hvsupport: Adapt to vbox driver rewrite

On 08/20/14 18:20, Michal Privoznik wrote: > Since vbox driver rewrite the virDriver structure init moved from > vbox_tmpl.c into vbox_common.c. However, our hvsupport.pl script > doesn't count with that. It still parses vbox_tmp.c and looks for > virDriver structure which is not found there anymore. As a result, > at hvsupport page is seems like vbox driver doesn't support > anything. > > Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com&gt; > --- > docs/hvsupport.pl | 2 +- > src/vbox/vbox_driver.c | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/docs/hvsupport.pl b/docs/hvsupport.pl > index f8483f9..34ba599 100755 > --- a/docs/hvsupport.pl > +++ b/docs/hvsupport.pl > @@ -28,7 +28,7 @@ my %groupheaders = ( > my @srcs; > find({ > wanted => sub { > - if (m!$srcdir/.*/\w+_(driver|tmpl|monitor|hal|udev)\.c$!) { > + if (m!$srcdir/.*/\w+_(driver|common|tmpl|monitor|hal|udev)\.c$!) { tmpl will need to be purged once the vbox driver rewrite is done

> push @srcs, $_ if $_ !~ /vbox_driver\.c/; > } > }, no_chdir => 1}, $srcdir); > diff --git a/src/vbox/vbox_driver.c b/src/vbox/vbox_driver.c > index f0c05fe..498be71 100644 > --- a/src/vbox/vbox_driver.c > +++ b/src/vbox/vbox_driver.c > @@ -210,5 +210,5 @@ static virDrvOpenStatus dummyConnectOpen(virConnectPtr conn, > static virDriver vboxDriverDummy = { > VIR_DRV_VBOX, > "VBOX", > - .connectOpen = dummyConnectOpen, > + .connectOpen = dummyConnectOpen, /* 0.6.3 */ > }; > ACK Peter