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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx>
> ---
>  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

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]