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
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list