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

NAKAMURA Takumi geek4civic at gmail.com
Mon Feb 10 02:59:33 PST 2014


Brad, sorry for the delay. Applied in llvm r201075 and clang r201076, thank you!

2014-01-29 6:54 GMT+09:00 Brad King <brad.king at kitware.com>:
> 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
>



More information about the llvm-commits mailing list