From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44714) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asQGQ-0006rW-JV for qemu-devel@nongnu.org; Tue, 19 Apr 2016 03:40:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1asQGM-0000Bf-R7 for qemu-devel@nongnu.org; Tue, 19 Apr 2016 03:40:02 -0400 Received: from m109-177.yeah.net ([123.58.177.109]:9118) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1asQGM-0000AU-0h for qemu-devel@nongnu.org; Tue, 19 Apr 2016 03:39:58 -0400 From: Yang Hongyang Date: Tue, 19 Apr 2016 15:39:13 +0800 Message-Id: <1461051553-14720-1-git-send-email-hongyang.yang@easystack.cn> Subject: [Qemu-devel] [PATCH] qemu-ga: do not run qga test when guest agent disabled List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Gerd Hoffmann , "Michael S. Tsirkin" , Michael Roth , Paolo Bonzini When configure with --disable-guest-agent, make check will fail with: ERROR:tests/test-qga.c:74:fixture_setup: assertion failed (error == NULL): Failed to execute child process "/home/xx/qemu/qemu-ga" (No such file or directory) (g-exec-error-quark, 8) make: *** [check-tests/test-qga] Error 1 This check was commented out by bab47d9a75a. I think that was by mistake, because the commit message of that commit didn't mention this change. Signed-off-by: Yang Hongyang Cc: Gerd Hoffmann Cc: Michael S. Tsirkin Cc: Michael Roth Cc: Paolo Bonzini --- tests/Makefile | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/Makefile b/tests/Makefile index 9de9598..9194f18 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -83,7 +83,9 @@ check-unit-y += tests/test-crypto-cipher$(EXESUF) check-unit-y += tests/test-crypto-secret$(EXESUF) check-unit-$(CONFIG_GNUTLS) += tests/test-crypto-tlscredsx509$(EXESUF) check-unit-$(CONFIG_GNUTLS) += tests/test-crypto-tlssession$(EXESUF) -#check-unit-$(CONFIG_LINUX) += tests/test-qga$(EXESUF) +ifneq (,$(findstring qemu-ga,$(TOOLS))) +check-unit-$(CONFIG_LINUX) += tests/test-qga$(EXESUF) +endif check-unit-y += tests/test-timed-average$(EXESUF) check-unit-y += tests/test-io-task$(EXESUF) check-unit-y += tests/test-io-channel-socket$(EXESUF) -- 1.8.3.1