Skip to content

Commit 629f184

Browse files
mizvekovalandefreitas
authored andcommitted
build: quote genexp for target_include_directories
LLVM_INCLUDE_DIRS can be a list of directories with more than one entry. Quote the generator expressions so that list expansion doesn't happen before it is parsed.
1 parent 3327505 commit 629f184

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

CMakeLists.txt

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -226,14 +226,14 @@ target_compile_definitions(
226226
# Dependencies
227227
target_include_directories(mrdocs-core
228228
SYSTEM PUBLIC
229-
$<BUILD_INTERFACE:${LLVM_INCLUDE_DIRS}>
230-
$<BUILD_INTERFACE:${CLANG_INCLUDE_DIRS}>
231-
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>
229+
"$<BUILD_INTERFACE:${LLVM_INCLUDE_DIRS}>"
230+
"$<BUILD_INTERFACE:${CLANG_INCLUDE_DIRS}>"
231+
"$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>"
232232
)
233233
target_include_directories(mrdocs-core
234234
SYSTEM PRIVATE
235-
$<BUILD_INTERFACE:${DUKTAPE_INCLUDE_DIRS}>
236-
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>
235+
"$<BUILD_INTERFACE:${DUKTAPE_INCLUDE_DIRS}>"
236+
"$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>"
237237
)
238238
target_link_libraries(mrdocs-core PRIVATE ${DUKTAPE_LIBRARY})
239239

0 commit comments

Comments
 (0)