[PATCH] Add configure option to disable the building of the API tests.

[PATCH] Add configure option to disable the building of the API tests.

[Conny Seidel]

When running the kvm unit tests on machines where some of the dependent
32bit devel libs aren't available, (libboost-pthread-mt, for example)
the build fails. The patch doesn't change the default behavior - it
simply adds the possibility to disable the API tests.

Usage:  ./configure --disable_api_tests

Signed-off-by: Conny Seidel <conny.seidel@amd.com>
---
 config-x86-common.mak |    2 ++
 configure             |    5 +++++
 2 files changed, 7 insertions(+), 0 deletions(-)

diff --git a/config-x86-common.mak b/config-x86-common.mak
index 033bae0..0b5ce71 100644
--- a/config-x86-common.mak
+++ b/config-x86-common.mak
@@ -36,8 +36,10 @@ tests-common = $(TEST_DIR)/vmexit.flat $(TEST_DIR)/tsc.flat \
                $(TEST_DIR)/kvmclock_test.flat  $(TEST_DIR)/eventinj.flat \
                $(TEST_DIR)/s3.flat
 
+ifndef NO_API
 tests-common += api/api-sample
 tests-common += api/dirty-log
+endif
 
 tests_and_config = $(TEST_DIR)/*.flat $(TEST_DIR)/unittests.cfg
 
diff --git a/configure b/configure
index efb8705..e7cf6dc 100755
--- a/configure
+++ b/configure
@@ -21,6 +21,7 @@ usage() {
 	    --ld=LD		   ld linker to use ($ld)
 	    --prefix=PREFIX        where to install things ($prefix)
 	    --kerneldir=DIR        kernel build directory for kvm.h ($kerneldir)
+	    --disable_api_tests    disables the build of the API tests
 EOF
     exit 1
 }
@@ -33,6 +34,9 @@ while [[ "$1" = -* ]]; do
 	opt="${opt%%=*}"
     fi
     case "$opt" in
+        --disable_api_tests)
+            api=true
+            ;;
 	--prefix)
 	    prefix="$arg"
 	    ;;
@@ -72,4 +76,5 @@ CC=$cross_prefix$cc
 LD=$cross_prefix$ld
 OBJCOPY=$cross_prefix$objcopy
 AR=$cross_prefix$ar
+NO_API=$api
 EOF
-- 
1.7.2.3

Re: [PATCH] Add configure option to disable the building of the API tests.

[Avi Kivity]

On 07/25/2011 03:39 PM, Conny Seidel wrote:
> When running the kvm unit tests on machines where some of the dependent
> 32bit devel libs aren't available, (libboost-pthread-mt, for example)
> the build fails. The patch doesn't change the default behavior - it
> simply adds the possibility to disable the API tests.

Please make ./configure autodetect the absence of the libraries and 
disable the tests automatically.

-- 
error compiling committee.c: too many arguments to function