From 39cff3dd6b5d9ebcf86f01e2c7e0bef62abd9d6f Mon Sep 17 00:00:00 2001 From: Alexander Tsoy Date: Thu, 25 Jun 2020 11:35:04 +0300 Subject: [PATCH 1/2] test: fix includetest to use CMAKE_MAKE_PROGRAM (#431) includetest currently fais if, for example, ninja is used as a CMake generator. Fix it by using CMAKE_MAKE_PROGRAM in the test. --- CMakeLists.txt | 2 +- test/includetest.sh | 7 ++++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/CMakeLists.txt b/CMakeLists.txt index 0b6228a2..290deb38 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -680,7 +680,7 @@ endif(LIBVNCSERVER_WITH_WEBSOCKETS) add_test(NAME cargs COMMAND test_cargstest) if(UNIX) - add_test(NAME includetest COMMAND ${TESTS_DIR}/includetest.sh ${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_INCLUDEDIR}) + add_test(NAME includetest COMMAND ${TESTS_DIR}/includetest.sh ${CMAKE_INSTALL_PREFIX}/${CMAKE_INSTALL_INCLUDEDIR} ${CMAKE_MAKE_PROGRAM}) endif(UNIX) if(FOUND_LIBJPEG_TURBO) add_test(NAME turbojpeg COMMAND test_tjunittest) diff --git a/test/includetest.sh b/test/includetest.sh index 23d602e6..6b064208 100755 --- a/test/includetest.sh +++ b/test/includetest.sh @@ -5,10 +5,11 @@ # expects install prefix like /usr as an argument PREFIX=$1 +CMAKE_MAKE_PROGRAM=$2 TMPDIR=$(mktemp -d) -make install DESTDIR=$TMPDIR +DESTDIR="$TMPDIR" $CMAKE_MAKE_PROGRAM install echo \ " @@ -19,6 +20,6 @@ int main() { return 0; } -" > $TMPDIR/includetest.c +" > "$TMPDIR"/includetest.c -cc -I $TMPDIR/$PREFIX $TMPDIR/includetest.c +cc -I "$TMPDIR/$PREFIX" "$TMPDIR"/includetest.c -- 2.26.2