Skip to content
Snippets Groups Projects
Commit 3686e38c authored by NAKAMURA Takumi's avatar NAKAMURA Takumi
Browse files

Fix Clang 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/Clang installation, we should
not ask CMake to manage permissions of that directory for us.  Instead,
give only our own include/clang and include/clang-c subdirectories to
the install(DIRECTORY) command.

Fixes PR4500. Patch by Brad King.

llvm-svn: 201076
parent c70006e1
No related branches found
No related tags found
No related merge requests found
...@@ -336,7 +336,7 @@ include_directories(BEFORE ...@@ -336,7 +336,7 @@ include_directories(BEFORE
) )
if (NOT LLVM_INSTALL_TOOLCHAIN_ONLY) if (NOT LLVM_INSTALL_TOOLCHAIN_ONLY)
install(DIRECTORY include/ install(DIRECTORY include/clang include/clang-c
DESTINATION include DESTINATION include
FILES_MATCHING FILES_MATCHING
PATTERN "*.def" PATTERN "*.def"
...@@ -345,7 +345,7 @@ if (NOT LLVM_INSTALL_TOOLCHAIN_ONLY) ...@@ -345,7 +345,7 @@ if (NOT LLVM_INSTALL_TOOLCHAIN_ONLY)
PATTERN ".svn" EXCLUDE PATTERN ".svn" EXCLUDE
) )
install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/include/ install(DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/include/clang
DESTINATION include DESTINATION include
FILES_MATCHING FILES_MATCHING
PATTERN "CMakeFiles" EXCLUDE PATTERN "CMakeFiles" EXCLUDE
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment