Fix test driver return value & disable nodeinfotest on non-linux (John Levon)

diff --git a/ChangeLog b/ChangeLog
index 638711bf075fd6f2377ea44343b582cc2d1fd443..2584bb01850fbffeb2ae46b221e5db0fcedb633f 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+Thu Dec 18 12:07:58 GMT 2008 Daniel P. Berrange <berrange@redhat.com>
+
+       * src/test.c: Fix error return value
+       * tests/nodeinfotest.c: Disable code compile completely on
+       non-linux (John Levon)
+
 Thu Dec 18 11:59:58 GMT 2008 Daniel P. Berrange <berrange@redhat.com>
 
        * qemud/remote.c: Fix unused variable compile warning when polkit

diff --git a/src/test.c b/src/test.c
index ed033420fe85fb46a7613f43ef68d42529f17d32..5d23dc108ad1c04acccffaa47255ef2a47d4adf0 100644 (file)
--- a/src/test.c
+++ b/src/test.c
@@ -1299,7 +1299,7 @@ static char *testGetOSType(virDomainPtr dom) {
 static unsigned long testGetMaxMemory(virDomainPtr domain) {
     testConnPtr privconn = domain->conn->privateData;
     virDomainObjPtr privdom;
-    unsigned long ret = -1;
+    unsigned long ret = 0;
 
     testDriverLock(privconn);
     privdom = virDomainFindByName(&privconn->domains,

diff --git a/tests/nodeinfotest.c b/tests/nodeinfotest.c
index ea7819a9350da59e896b39abea23a368b4e85100..ba3622f241c214fdb178e6fa1ddc6f83428772c0 100644 (file)
--- a/tests/nodeinfotest.c
+++ b/tests/nodeinfotest.c
@@ -10,6 +10,16 @@
 #include "nodeinfo.h"
 #include "util.h"
 
+#ifndef __linux__
+
+static int
+mymain(int argc, char **argv)
+{
+    exit (77); /* means 'test skipped' for automake */
+}
+
+#else
+
 static char *progname;
 static char *abs_srcdir;
 
@@ -78,9 +88,6 @@ mymain(int argc, char **argv)
         "nodeinfo-6",
     };
     char cwd[PATH_MAX];
-#ifndef __linux__
-    exit (77); /* means 'test skipped' for automake */
-#endif
 
     abs_srcdir = getenv("abs_srcdir");
     if (!abs_srcdir)
@@ -102,4 +109,7 @@ mymain(int argc, char **argv)
     return(ret==0 ? EXIT_SUCCESS : EXIT_FAILURE);
 }
 
+#endif /* __linux__ */
+
 VIRT_TEST_MAIN(mymain)
+