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

[Greg Fitzgerald]

LGTM, thanks!

On Tue, Jan 28, 2014 at 1:54 PM, Brad King <brad.king at kitware.com> wrote:
> 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
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>