From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54748) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WT8Dr-0003tQ-Cr for qemu-devel@nongnu.org; Thu, 27 Mar 2014 07:11:53 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WT8Dl-0002B8-9f for qemu-devel@nongnu.org; Thu, 27 Mar 2014 07:11:47 -0400 Received: from mx1.redhat.com ([209.132.183.28]:13221) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WT8Dl-0002B1-00 for qemu-devel@nongnu.org; Thu, 27 Mar 2014 07:11:41 -0400 From: Stefan Hajnoczi Date: Thu, 27 Mar 2014 12:11:26 +0100 Message-Id: <1395918686-27043-3-git-send-email-stefanha@redhat.com> In-Reply-To: <1395918686-27043-1-git-send-email-stefanha@redhat.com> References: <1395918686-27043-1-git-send-email-stefanha@redhat.com> Subject: [Qemu-devel] [PATCH 2/2] tests: correctly skip qtest on non-POSIX hosts List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Paolo Bonzini , Andreas Faerber , Stefan Hajnoczi , Stefan Weil qtest test cases only work on POSIX hosts. The following line only defines dependencies for qtest binaries on POSIX hosts: check-qtest-$(CONFIG_POSIX)=$(foreach TARGET,$(TARGETS),$(check-qtest-$(TARGET)-y)) But the QTEST_TARGETS definition earlier in the Makefile fails to check CONFIG_POSIX. The causes targets to be generated for qtest test cases even though we don't know how to build the binaries. The following error message is printed when trying to run gtester on a binary that was never built: GLib-WARNING **: Failed to execute test binary: tests/endianness-test.exe: Failed to execute child process "tests/endianness-test.exe" (No such file or directory) This patch makes QTEST_TARGETS empty on non-POSIX hosts. This prevents the targets from being generated. Signed-off-by: Stefan Hajnoczi --- tests/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/Makefile b/tests/Makefile index 803c8e6..e1960c1 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -277,7 +277,8 @@ tests/qemu-iotests/socket_scm_helper$(EXESUF): tests/qemu-iotests/socket_scm_hel # QTest rules TARGETS=$(patsubst %-softmmu,%, $(filter %-softmmu,$(TARGET_DIRS))) -QTEST_TARGETS=$(foreach TARGET,$(TARGETS), $(if $(check-qtest-$(TARGET)-y), $(TARGET),)) +QTEST_TARGETS=$(foreach TARGET,$(TARGETS), \ + $(if $(check-qtest-$(TARGET)-$(CONFIG_POSIX)), $(TARGET),)) check-qtest-$(CONFIG_POSIX)=$(foreach TARGET,$(TARGETS), $(check-qtest-$(TARGET)-y)) qtest-obj-y = tests/libqtest.o libqemuutil.a libqemustub.a -- 1.8.5.3