From 86a12671c5d33063b6f958bdcca7c9d14cd5aac8 Mon Sep 17 00:00:00 2001 From: Michal Orzel Date: Wed, 4 Jun 2025 09:21:28 +0200 Subject: [PATCH] tests/vpci: Use $(CC) instead of $(HOSTCC) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Depending on the build environment, HOSTCC can be different than CC. With the recent `install` rule addition, this would put a binary of a wrong format in the destdir (e.g. building tests on x86 host for Arm target). Take the opportunity to adjust the `run` rule to only run the test if HOSTCC is CC, else print a warning message. Fixes: 96a587a05736 ("tools/tests: Add install target for vPCI") Signed-off-by: Michal Orzel Reviewed-by: Roger Pau Monné --- tools/tests/vpci/Makefile | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tools/tests/vpci/Makefile b/tools/tests/vpci/Makefile index 9450f7593a..97359ff67f 100644 --- a/tools/tests/vpci/Makefile +++ b/tools/tests/vpci/Makefile @@ -8,10 +8,14 @@ all: $(TARGET) .PHONY: run run: $(TARGET) +ifeq ($(CC),$(HOSTCC)) ./$(TARGET) +else + $(warning HOSTCC != CC, will not run test) +endif $(TARGET): vpci.c vpci.h list.h main.c emul.h - $(HOSTCC) $(CFLAGS_xeninclude) -g -o $@ vpci.c main.c + $(CC) $(CFLAGS_xeninclude) -g -o $@ vpci.c main.c .PHONY: clean clean: