[libvirt] [PATCH] libvirtd: Don't check WITH_LIBVIRTD in C source files

Hu Tao hutao at cn.fujitsu.com
Tue Feb 5 05:34:09 UTC 2013


Please add one more line to commit msg:

On Tue, Feb 05, 2013 at 10:20:37AM +0800, Hu Tao wrote:
> Don't check WITH_LIBVIRTD in C source files because we will build
> the sources even without libvirtd.

  This patch also fix the build failure when buliding with
  --without-libvirtd.

> ---
>  src/driver.h               | 4 ----
>  src/libvirt.c              | 7 -------
>  src/libvirt_internal.h     | 2 --
>  src/remote/remote_driver.c | 6 ------
>  src/xen/xen_driver.c       | 6 +-----
>  5 files changed, 1 insertion(+), 24 deletions(-)
> 
> diff --git a/src/driver.h b/src/driver.h
> index 02ddd83..dab7495 100644
> --- a/src/driver.h
> +++ b/src/driver.h
> @@ -1512,7 +1512,6 @@ struct _virStorageDriver {
>      virDrvStoragePoolIsPersistent           poolIsPersistent;
>  };
>  
> -# ifdef WITH_LIBVIRTD
>  
>  typedef int (*virDrvStateInitialize) (bool privileged,
>                                        virStateInhibitCallback callback,
> @@ -1531,7 +1530,6 @@ struct _virStateDriver {
>      virDrvStateReload      reload;
>      virDrvStateStop        stop;
>  };
> -# endif
>  
>  
>  typedef struct _virDeviceMonitor virDeviceMonitor;
> @@ -1768,9 +1766,7 @@ int virRegisterStorageDriver(virStorageDriverPtr);
>  int virRegisterDeviceMonitor(virDeviceMonitorPtr);
>  int virRegisterSecretDriver(virSecretDriverPtr);
>  int virRegisterNWFilterDriver(virNWFilterDriverPtr);
> -# ifdef WITH_LIBVIRTD
>  int virRegisterStateDriver(virStateDriverPtr);
> -# endif
>  void virDriverModuleInitialize(const char *defmoddir);
>  void *virDriverLoadModule(const char *name);
>  
> diff --git a/src/libvirt.c b/src/libvirt.c
> index f81a3de..e2a7b5d 100644
> --- a/src/libvirt.c
> +++ b/src/libvirt.c
> @@ -118,10 +118,8 @@ static virSecretDriverPtr virSecretDriverTab[MAX_DRIVERS];
>  static int virSecretDriverTabCount = 0;
>  static virNWFilterDriverPtr virNWFilterDriverTab[MAX_DRIVERS];
>  static int virNWFilterDriverTabCount = 0;
> -#ifdef WITH_LIBVIRTD
>  static virStateDriverPtr virStateDriverTab[MAX_DRIVERS];
>  static int virStateDriverTabCount = 0;
> -#endif
>  
>  
>  #if defined(POLKIT_AUTH)
> @@ -771,7 +769,6 @@ virRegisterDriver(virDriverPtr driver)
>      return virDriverTabCount++;
>  }
>  
> -#ifdef WITH_LIBVIRTD
>  /**
>   * virRegisterStateDriver:
>   * @driver: pointer to a driver block
> @@ -885,10 +882,6 @@ int virStateStop(void) {
>      return ret;
>  }
>  
> -#endif
> -
> -
> -
>  /**
>   * virGetVersion:
>   * @libVer: return value for the library version (OUT)
> diff --git a/src/libvirt_internal.h b/src/libvirt_internal.h
> index 595d2db..b82f6b9 100644
> --- a/src/libvirt_internal.h
> +++ b/src/libvirt_internal.h
> @@ -27,7 +27,6 @@
>  
>  # include "internal.h"
>  
> -# ifdef WITH_LIBVIRTD
>  typedef void (*virStateInhibitCallback)(bool inhibit,
>                                          void *opaque);
>  
> @@ -37,7 +36,6 @@ int virStateInitialize(bool privileged,
>  int virStateCleanup(void);
>  int virStateReload(void);
>  int virStateStop(void);
> -# endif
>  
>  /* Feature detection.  This is a libvirt-private interface for determining
>   * what features are supported by the driver.
> diff --git a/src/remote/remote_driver.c b/src/remote/remote_driver.c
> index 341321b..354a216 100644
> --- a/src/remote/remote_driver.c
> +++ b/src/remote/remote_driver.c
> @@ -156,7 +156,6 @@ static void remoteDomainEventQueue(struct private_data *priv, virDomainEventPtr
>  /* Helper functions for remoteOpen. */
>  static char *get_transport_from_scheme(char *scheme);
>  
> -#ifdef WITH_LIBVIRTD
>  static int
>  remoteStartup(bool privileged ATTRIBUTE_UNUSED,
>                virStateInhibitCallback callback ATTRIBUTE_UNUSED,
> @@ -168,7 +167,6 @@ remoteStartup(bool privileged ATTRIBUTE_UNUSED,
>      inside_daemon = true;
>      return 0;
>  }
> -#endif
>  
>  #ifndef WIN32
>  /**
> @@ -6382,12 +6380,10 @@ static virNWFilterDriver nwfilter_driver = {
>  };
>  
>  
> -#ifdef WITH_LIBVIRTD
>  static virStateDriver state_driver = {
>      .name = "Remote",
>      .initialize = remoteStartup,
>  };
> -#endif
>  
>  
>  /** remoteRegister:
> @@ -6408,9 +6404,7 @@ remoteRegister(void)
>      if (virRegisterDeviceMonitor(&dev_monitor) == -1) return -1;
>      if (virRegisterSecretDriver(&secret_driver) == -1) return -1;
>      if (virRegisterNWFilterDriver(&nwfilter_driver) == -1) return -1;
> -#ifdef WITH_LIBVIRTD
>      if (virRegisterStateDriver(&state_driver) == -1) return -1;
> -#endif
>  
>      return 0;
>  }
> diff --git a/src/xen/xen_driver.c b/src/xen/xen_driver.c
> index 2795ebc..d65f4a6 100644
> --- a/src/xen/xen_driver.c
> +++ b/src/xen/xen_driver.c
> @@ -91,7 +91,7 @@ static struct xenUnifiedDriver const * const drivers[XEN_UNIFIED_NR_DRIVERS] = {
>  #endif
>  };
>  
> -#if defined WITH_LIBVIRTD || defined __sun
> +#ifdef __sun
>  static bool inside_daemon = false;
>  #endif
>  
> @@ -200,7 +200,6 @@ done:
>      return res;
>  }
>  
> -#ifdef WITH_LIBVIRTD
>  
>  static int
>  xenInitialize(bool privileged ATTRIBUTE_UNUSED,
> @@ -216,7 +215,6 @@ static virStateDriver state_driver = {
>      .initialize = xenInitialize,
>  };
>  
> -#endif
>  
>  /*----- Dispatch functions. -----*/
>  
> @@ -2398,9 +2396,7 @@ static virDriver xenUnifiedDriver = {
>  int
>  xenRegister(void)
>  {
> -#ifdef WITH_LIBVIRTD
>      if (virRegisterStateDriver(&state_driver) == -1) return -1;
> -#endif
>  
>      return virRegisterDriver(&xenUnifiedDriver);
>  }
> -- 
> 1.8.0.1.240.ge8a1f5a
> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list




More information about the libvir-list mailing list