[llvm] r201075 - Fix LLVM install rules to not set permissions on include/
NAKAMURA Takumi
geek4civic at gmail.com
Mon Feb 10 02:50:55 PST 2014
Author: chapuni
Date: Mon Feb 10 04:50:55 2014
New Revision: 201075
URL: http://llvm.org/viewvc/llvm-project?rev=201075&view=rev
Log:
Fix LLVM install rules to not set permissions on include/
The CMake install(DIRECTORY) command documents that it sets permissions
on directories it is asked to install. Since the <prefix>/include
directory may not be exclusive to the LLVM installation, we should not
ask CMake to manage permissions of that directory for us. Instead, give
only our own include/llvm and include/llvm-c subdirectories to the
install(DIRECTORY) command.
Fixes PR4500. Patch by Brad King.
Modified:
llvm/trunk/CMakeLists.txt
Modified: llvm/trunk/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/CMakeLists.txt?rev=201075&r1=201074&r2=201075&view=diff
==============================================================================
--- llvm/trunk/CMakeLists.txt (original)
+++ llvm/trunk/CMakeLists.txt Mon Feb 10 04:50:55 2014
@@ -517,7 +517,7 @@ endif()
add_subdirectory(cmake/modules)
if (NOT LLVM_INSTALL_TOOLCHAIN_ONLY)
- install(DIRECTORY include/
+ install(DIRECTORY include/llvm include/llvm-c
DESTINATION include
FILES_MATCHING
PATTERN "*.def"
@@ -528,7 +528,7 @@ if (NOT LLVM_INSTALL_TOOLCHAIN_ONLY)
PATTERN ".svn" EXCLUDE
)
- install(DIRECTORY ${LLVM_INCLUDE_DIR}/
+ install(DIRECTORY ${LLVM_INCLUDE_DIR}/llvm
DESTINATION include
FILES_MATCHING
PATTERN "*.def"
More information about the llvm-commits
mailing list