[PATCH] D11682: [libcxxabi] Add "install-libcxxabi" target.

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 18 16:40:00 PDT 2015


EricWF added a comment.

Followup on @beanz comments.


================
Comment at: src/CMakeLists.txt:123
@@ -124,1 +122,3 @@
+  LIBRARY DESTINATION lib${LIBCXXABI_LIBDIR_SUFFIX} COMPONENT libcxxabi
+  ARCHIVE DESTINATION lib${LIBCXXABI_LIBDIR_SUFFIX} COMPONENT libcxxabi
   )
----------------
beanz wrote:
> You should not specify both ARCHIVE and LIBRARY in the same install command. One of the two will be ignored. As per the documentation (http://www.cmake.org/cmake/help/v3.0/command/install.html).
> 
> You will need to do something similar to what is done in AddLLVM.cmake:507-520.
I don't see where in the documentation that it says you can't use ARCHIVE and LIBRARY in the same install command. Could you be more explicit?


http://reviews.llvm.org/D11682





More information about the cfe-commits mailing list