From d25265fee1d3fa4d4469f94b2c1ae45bb803ba9e Mon Sep 17 00:00:00 2001 From: Martchus Date: Mon, 23 Oct 2017 01:08:25 +0200 Subject: [PATCH] Don't add include path for config.h to header-only libs This causes CMake errors and CMake is acutally right, it makes no sense because the config header is only available during build but a header-only lib is not being built. --- cmake/modules/ConfigHeader.cmake | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/cmake/modules/ConfigHeader.cmake b/cmake/modules/ConfigHeader.cmake index 4669721..7b262ee 100644 --- a/cmake/modules/ConfigHeader.cmake +++ b/cmake/modules/ConfigHeader.cmake @@ -55,8 +55,10 @@ configure_file( ) # ensure generated include files can be included via #include "resources/config.h" -foreach(TARGET_NAME ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX} ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX}_static ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX}_tests ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX}_testlib) - if(TARGET ${TARGET_NAME}) - target_include_directories(${TARGET_NAME} PRIVATE "${CMAKE_CURRENT_BINARY_DIR}") - endif() -endforeach() +if(NOT META_HEADER_ONLY_LIB) + foreach(TARGET_NAME ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX} ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX}_static ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX}_tests ${TARGET_PREFIX}${META_PROJECT_NAME}${TARGET_SUFFIX}_testlib) + if(TARGET ${TARGET_NAME}) + target_include_directories(${TARGET_NAME} PRIVATE "${CMAKE_CURRENT_BINARY_DIR}") + endif() + endforeach() +endif()