[PATCH] Fix PR4500 - Do not set permissions on include/

[Brad King]

Hi Folks,

On 01/28/2014 03:51 PM, Chandler Carruth wrote:
> Can you post it to the commit list for review?

Here are patches to address PR4500 for LLVM and Clang:

 http://llvm.org/bugs/show_bug.cgi?id=4500

The problem is that we were using the CMake install(DIRECTORY) command
to manage the top-level "include/" directory under the install prefix,
and that command sets permissions on the directories it installs.

These patches tell CMake to only install the

 include/{llvm,llvm-c,clang,clang-c}

directories and avoid trying to set permissions any higher than that.

See the commit message in each patch for details.

Thanks,
-Brad
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-CMake-Fix-LLVM-install-rules-to-not-set-permissions-.patch
Type: text/x-diff
Size: 1423 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140128/367b4a6e/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-CMake-Fix-Clang-install-rules-to-not-set-permissions.patch
Type: text/x-diff
Size: 1471 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140128/367b4a6e/attachment-0001.patch>