[libcxx] r178579 - The cmake script is failing to copy cxxabi.h to the right place because it was generating to destination path like so /include// and dstdir can legally be blank from my interpretation of the script, and this would then generate a path like libcxx/include// which is illegal.

Howard Hinnant hhinnant at apple.com
Tue Apr 2 14:33:01 PDT 2013


Author: hhinnant
Date: Tue Apr  2 16:33:01 2013
New Revision: 178579

URL: http://llvm.org/viewvc/llvm-project?rev=178579&view=rev
Log:
The cmake script is failing to copy cxxabi.h to the right place because it was generating to destination path like so /include// and dstdir can legally be blank from my interpretation of the script, and this would then generate a path like libcxx/include// which is illegal.

Modified:
    libcxx/trunk/CMakeLists.txt

Modified: libcxx/trunk/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/CMakeLists.txt?rev=178579&r1=178578&r2=178579&view=diff
==============================================================================
--- libcxx/trunk/CMakeLists.txt (original)
+++ libcxx/trunk/CMakeLists.txt Tue Apr  2 16:33:01 2013
@@ -106,7 +106,7 @@ macro(setup_abi_lib abipathvar abidefine
           OUTPUT "${CMAKE_BINARY_DIR}/include/${dstdir}/${ifile}"
           COMMAND ${CMAKE_COMMAND} -E copy_if_different
             "${incpath}/${fpath}"
-            "${CMAKE_BINARY_DIR}/include/${dstdir}/"
+            "${CMAKE_BINARY_DIR}/include/${dstdir}"
           MAIN_DEPENDENCY "${incpath}/${fpath}"
           )
         list(APPEND LIBCXX_CXX_ABI_DEPS





More information about the cfe-commits mailing list