Re: [PATCH] fix compilation with USE_XEN == 0

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

 



"Richard W.M. Jones" <rjones@xxxxxxxxxx> wrote:
> Guido Guenther wrote:
>> On Thu, Jan 31, 2008 at 04:14:09PM +0100, Jim Meyering wrote:
>>> Guido Guenther <agx@xxxxxxxxxxx> wrote:
>>>> statstest.c needs to check if USE_XEN > 0, otherwise compilation breaks
>>>> on --without-xen builds. Please apply if appropriate. Patch is against
...
> Looks good to me.

Same here.
This is what I expect to apply in an hour or so:

Thu Jan 31 20:12:40 CET 2008  Guido Guenther  <agx@xxxxxxxxxxx>

	Avoid compilation failure when building --without-xen.
	* tests/statstest.c (testDevice) [!WITH_XEN]: #ifdef-out.
	(main) [!WITH_XEN]: #ifdef-out the body.

Signed-off-by: Jim Meyering <meyering@xxxxxxxxxx>
---
 ChangeLog         |    6 ++++++
 tests/statstest.c |    4 ++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 27e5089..1c46da7 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+Thu Jan 31 20:12:40 CET 2008  Guido Guenther  <agx@xxxxxxxxxxx>
+
+	Avoid compilation failure when building --without-xen.
+	* tests/statstest.c (testDevice) [!WITH_XEN]: #ifdef-out.
+	(main) [!WITH_XEN]: #ifdef-out the body.
+
 Wed Jan 30 20:49:34 CET 2008  Jim Meyering  <meyering@xxxxxxxxxx>

 	Avoid virsh leaks due to missing virDomainFree(dom) calls
diff --git a/tests/statstest.c b/tests/statstest.c
index 4c19833..6213da8 100644
--- a/tests/statstest.c
+++ b/tests/statstest.c
@@ -12,7 +12,7 @@ static void testQuietError(void *userData ATTRIBUTE_UNUSED, virErrorPtr error AT
     /* nada */
 }

-#ifdef __linux__
+#if __linux__ && WITH_XEN
 static int testDevice(const char *path, int expect)
 {
     int actual = xenLinuxDomainDeviceID(NULL, 1, path);
@@ -31,7 +31,7 @@ int
 main(void)
 {
     int ret = 0;
-#ifdef __linux__
+#if __linux__ && WITH_XEN
     /* Some of our tests delibrately test failure cases, so
      * register a handler to stop error messages cluttering
      * up display
--
1.5.4.rc5.1.ge6bfe

--
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]