[libvirt] [PATCH] make consistent assumptions about libvirtd dependence

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

 



Hi -
  Currently src/Makefile.am says that only the QEMU, LXC, and storage
drivers are dependent on libvirtd.  But the driver registration code in
virInitialize wasn't registering the remote or openVz driver when
WITH_LIBVIRTD was not defined.  The attached patch makes the
virInitialize code consistent with the assumptions stated in
src/Makefile.am.
  But note I'm assuming the assumptions in src/Makefile.am are correct.
I'm not sure what's really dependent on libvirtd, so Someone Who Knows
Better should verify this.

Thanks,
Dave

commit 24d5708877fce90c4e85e3b2ede07b74f4a98224
Author: David Lively <dlively@xxxxxxxxxxxxxxx>
Date:   Thu Sep 11 11:56:20 2008 -0400

    Make driver registration assume the same libvirtd dependences
    expressed in src/Makefile.am (i.e., that only the QEMU, LXC, and storage
    drivers depend on libvirtd).

diff --git a/src/libvirt.c b/src/libvirt.c
index 54ed8cf..3d66f5a 100644
--- a/src/libvirt.c
+++ b/src/libvirt.c
@@ -286,20 +286,20 @@ virInitialize(void)
 #ifdef WITH_XEN
     if (xenUnifiedRegister () == -1) return -1;
 #endif
-#ifdef WITH_LIBVIRTD
-#ifdef WITH_QEMU
-    if (qemudRegister() == -1) return -1;
+#ifdef WITH_REMOTE
+    if (remoteRegister () == -1) return -1;
 #endif
 #ifdef WITH_OPENVZ
     if (openvzRegister() == -1) return -1;
 #endif
+#ifdef WITH_LIBVIRTD
+#ifdef WITH_QEMU
+    if (qemudRegister() == -1) return -1;
+#endif
 #ifdef WITH_LXC
     if (lxcRegister() == -1) return -1;
 #endif
     if (storageRegister() == -1) return -1;
-#ifdef WITH_REMOTE
-    if (remoteRegister () == -1) return -1;
-#endif
 #endif
 
     return(0);
--
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]