[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add
Petr Hosek via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 20 09:04:07 PST 2017
phosek created this revision.
Herald added a subscriber: mgorny.
Escaping ; in list arguments passed to ExternalProject_Add doesn't seem
to be working in newer versions of CMake (see
https://public.kitware.com/Bug/view.php?id=16137 for more details). Use
a custom LIST_SEPARATOR instead which is the officially supported way.
Repository:
rL LLVM
https://reviews.llvm.org/D40257
Files:
CMakeLists.txt
Index: CMakeLists.txt
===================================================================
--- CMakeLists.txt
+++ CMakeLists.txt
@@ -669,7 +669,7 @@
if("${${variableName}}" STREQUAL "")
set(value "")
else()
- string(REPLACE ";" "\;" value ${${variableName}})
+ string(REPLACE ";" "," value ${${variableName}})
endif()
list(APPEND PASSTHROUGH_VARIABLES
-D${variableName}=${value})
@@ -697,6 +697,7 @@
USES_TERMINAL_CONFIGURE 1
USES_TERMINAL_BUILD 1
USES_TERMINAL_INSTALL 1
+ LIST_SEPARATOR ,
)
# exclude really-install from main target
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40257.123605.patch
Type: text/x-patch
Size: 622 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20171120/4d8419f0/attachment.bin>
More information about the cfe-commits
mailing list