EricWF added inline comments.

================
Comment at: CMakeLists.txt:105
@@ +104,3 @@
+if (${CMAKE_SYSTEM_NAME} MATCHES "Darwin" AND NOT 
LIBCXX_OVERRIDE_DARWIN_INSTALL)
+  if ("${CMAKE_INSTALL_PREFIX}" STREQUAL "/usr")
+    message(WARNING "Disabling libc++ install rules because installation would 
"
----------------
jroelofs wrote:
> I would check for "/usr/" too.
CMake canonicalizes `CMAKE_INSTALL_PREFIX` which includes removing the trailing 
slash. I tested this before submitting the patch.


http://reviews.llvm.org/D12209



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

Reply via email to