From 092eb22107b554c41bb7fd9f36b87e33d03a214b Mon Sep 17 00:00:00 2001 From: Jim Harris Date: Wed, 21 Nov 2018 01:45:04 -0700 Subject: [PATCH] build: don't include spdk.app.mk in test/cpp_headers This Makefile doesn't depend on anything in spdk.app.mk, and provides its own rules that are different than any of the other applications (including unit tests). The exception here is the simple "install : app" rule. Just add that rule to the cpp_headers Makefile instead of including spdk.app.mk just for that rule. Signed-off-by: Jim Harris Change-Id: I684dfba9108dfa7f06d7eb4acecd6806e5797312 Reviewed-on: https://review.gerrithub.io/434282 Tested-by: SPDK CI Jenkins Chandler-Test-Pool: SPDK Automated Test System Reviewed-by: Darek Stojaczyk Reviewed-by: Ben Walker --- test/cpp_headers/Makefile | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/test/cpp_headers/Makefile b/test/cpp_headers/Makefile index cd159bb58..0b8d3bd05 100644 --- a/test/cpp_headers/Makefile +++ b/test/cpp_headers/Makefile @@ -33,11 +33,12 @@ SPDK_ROOT_DIR := $(abspath $(CURDIR)/../..) include $(SPDK_ROOT_DIR)/mk/spdk.common.mk -include $(SPDK_ROOT_DIR)/mk/spdk.app.mk HEADERS := $(wildcard $(SPDK_ROOT_DIR)/include/spdk/*.h) CXX_SRCS := $(patsubst %.h,%.cpp,$(notdir $(HEADERS))) +install : all + %.cpp: $(SPDK_ROOT_DIR)/include/spdk/%.h $(Q)echo " TEST_HEADER include/spdk/$(notdir $<)"; \ echo '#include "spdk/$(notdir $<)"' > $@