[clang][cmake+shared] rpath install name consistency

David Fang fang at csl.cornell.edu
Thu Feb 27 15:15:04 PST 2014


Rafael,
 	I believe that you can revert r202302 back to use @rpath, which 
should work on darwin9+, and then I'll post a patch to use @loader_path 
for darwin8-only (or likely keep it local my darwin8 branch).

David

> On Feb 27, 2014, at 2:15 PM, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
>
>> Argyrios, note that this changes the result from @rpath to @excutable_path.
>
> No, that?s bad, libclang needs to use @rpath.
>
>>
>> On 27 February 2014 17:13, David Fang <fang at csl.cornell.edu> wrote:
>>> Hi,
>>>        This patch makes linking libclang consistent w.r.t. RPATH install
>>> name, as set by LLVM's CMakeLists.txt (under
>>> CMAKE_BUILD_WITH_INSTALL_RPATH), which was updated by Rafael at r202302.
>>> Does this seem ok?
>>>
>>> David
>>>
>>>
>>> --
>>> David Fang
>>> http://www.csl.cornell.edu/~fang/
>

-- 
David Fang
http://www.csl.cornell.edu/~fang/


More information about the cfe-commits mailing list