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

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

 



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
current cvs.
Thanks for the report, but your patch didn't make it to the list.
That has happened to me when I've included the output
of e.g., git format-patch in the message.  Its headers
confuse the old version of Mailman that this list is using.

If you think something like that happened here, please
send me a copy of your original message and I'll look into it.
Yes, I actually used git. Here it is without the headers:

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

Looks good to me.

Rich.

--
Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/
Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod
Street, Windsor, Berkshire, SL4 1TE, United Kingdom.  Registered in
England and Wales under Company Registration No. 03798903

Attachment: smime.p7s
Description: S/MIME Cryptographic 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]