[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add

2017-11-28 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL319264: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add (authored by phosek). Changed prior to commit: https://reviews.llvm.org/D40257?vs=123710=124663#toc Repository: rL LLVM

[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add

2017-11-27 Thread Chris Bieneman via Phabricator via cfe-commits
beanz accepted this revision. beanz added a comment. This revision is now accepted and ready to land. LGTM! Repository: rL LLVM https://reviews.llvm.org/D40257 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add

2017-11-27 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. Can you please take a look at this one as well (this one is Clang, the other one was LLVM)? Repository: rL LLVM https://reviews.llvm.org/D40257 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add

2017-11-20 Thread Petr Hosek via Phabricator via cfe-commits
phosek updated this revision to Diff 123710. Repository: rL LLVM https://reviews.llvm.org/D40257 Files: CMakeLists.txt Index: CMakeLists.txt === --- CMakeLists.txt +++ CMakeLists.txt @@ -653,7 +653,7 @@

[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add

2017-11-20 Thread Petr Hosek via Phabricator via cfe-commits
phosek updated this revision to Diff 123640. Repository: rL LLVM https://reviews.llvm.org/D40257 Files: CMakeLists.txt Index: CMakeLists.txt === --- CMakeLists.txt +++ CMakeLists.txt @@ -653,7 +653,7 @@

[PATCH] D40257: [CMake] Use LIST_SEPARATOR rather than escaping in ExternalProject_Add

2017-11-20 Thread Petr Hosek via Phabricator via cfe-commits
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