[libvirt] [PATCH 27/49] daemon: Fix the wrong macro name

Guannan Ren gren at redhat.com
Tue Jul 24 14:05:43 UTC 2012


On 07/20/2012 10:25 PM, Osier Yang wrote:
> WITH_INTERFACE is not defined, it should be WITH_NETCF there to load
> the interface driver.
>
> Eric posted patch weeks ago to resolve the problems in the whole
> build system, but it's not finalised yet:
>
> https://www.redhat.com/archives/libvir-list/2012-June/msg01299.html
>
> I'm going to simply fix the wrong macro name here so that the
> interface driver could loaded, and continue the work on the listing
> API for interface driver.
> ---
>   daemon/libvirtd.c           |    2 +-
>   tests/virdrivermoduletest.c |    2 +-
>   2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/daemon/libvirtd.c b/daemon/libvirtd.c
> index 8c434a0..6dca9b0 100644
> --- a/daemon/libvirtd.c
> +++ b/daemon/libvirtd.c
> @@ -378,7 +378,7 @@ static void daemonInitialize(void)
>   # ifdef WITH_NWFILTER
>       virDriverLoadModule("nwfilter");
>   # endif
> -# ifdef WITH_INTERFACE
> +# ifdef WITH_NETCF
>       virDriverLoadModule("interface");
>   # endif
>   # ifdef WITH_QEMU
> diff --git a/tests/virdrivermoduletest.c b/tests/virdrivermoduletest.c
> index 405574e..d1fd649 100644
> --- a/tests/virdrivermoduletest.c
> +++ b/tests/virdrivermoduletest.c
> @@ -79,7 +79,7 @@ mymain(void)
>   #ifdef WITH_NWFILTER
>       TEST("nwfilter", NULL);
>   #endif
> -#ifdef WITH_INTERFACE
> +#ifdef WITH_NETCF
>       TEST("interface", NULL);
>   #endif
>   #ifdef WITH_QEMU

     ACK, I need this patch, so I tested it, it's fine.




More information about the libvir-list mailing list