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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to