External/fmt
8
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

cmake: treat public includes as system includes to avoid warning from headers

Browse Source 
Include directories for target fmt was splitted into INTERFACE and PRIVATE.
INTERFACE includes was marked as SYSTEM to suppress warnings in targets that
are linking fmt:fmt. PRIVATE includes are left as normal includes so that
warnings still occur when building fmt.

Includes for fmt-header-only are now marked as SYSTEM.
This commit is contained in:
Jan Schwers 2020-09-03 12:32:35 +02:00
parent 6cccdc24bc
commit f9fac58eb9
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
CMakeLists.txt
View File

@ -211,7 +211,9 @@ endif ()
target_compile_features(fmt INTERFACE ${FMT_REQUIRED_FEATURES})
target_include_directories(fmt PUBLIC
target_include_directories(fmt PRIVATE ${PROJECT_SOURCE_DIR}/include)
target_include_directories(fmt SYSTEM INTERFACE
$<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>
$<INSTALL_INTERFACE:${FMT_INC_DIR}>)
@ -246,7 +248,7 @@ add_library(fmt::fmt-header-only ALIAS fmt-header-only)
target_compile_definitions(fmt-header-only INTERFACE FMT_HEADER_ONLY=1)
target_compile_features(fmt-header-only INTERFACE ${FMT_REQUIRED_FEATURES})
target_include_directories(fmt-header-only INTERFACE
target_include_directories(fmt-header-only SYSTEM INTERFACE
$<BUILD_INTERFACE:${PROJECT_SOURCE_DIR}/include>
$<INSTALL_INTERFACE:${FMT_INC_DIR}>)