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

David Fang fang at csl.cornell.edu
Thu Feb 27 14:13:35 PST 2014


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/
-------------- next part --------------
commit 267c192d4705af456db4b64c0ac89f9a0f87947e
Author: David Fang <fang at csl.cornell.edu>
Date:   Thu Feb 27 14:07:31 2014 -0800

    for consistency, use RPATH install name dir set by LLVM's cmake file

diff --git a/tools/libclang/CMakeLists.txt b/tools/libclang/CMakeLists.txt
index a30e5da..d6bbbf8 100644
--- a/tools/libclang/CMakeLists.txt
+++ b/tools/libclang/CMakeLists.txt
@@ -105,8 +105,5 @@ if(ENABLE_SHARED)
 
     set_property(TARGET libclang APPEND_STRING PROPERTY
                  LINK_FLAGS ${LIBCLANG_LINK_FLAGS})
-    set_target_properties(libclang
-      PROPERTIES
-      INSTALL_NAME_DIR "@rpath")
   endif()
 endif()


More information about the cfe-commits mailing list