labath added a comment.
Yeah, not requiring LD_LIBRARY_PATH in the BUILD_SHARED_LIBS build would be
nice. The fix isn't particularly pretty, but I don't know if there is a better
way to do that (besides the improvements in inline comments).
================
Comment at: lldb/source/API/CMakeLists.txt:117
+# loaded from Python.
+if(LLDB_ENABLE_PYTHON AND BUILD_SHARED_LIBS AND UNIX)
+ if(LLVM_INSTALL_PREFIX AND NOT (LLVM_INSTALL_PREFIX STREQUAL
CMAKE_INSTALL_PREFIX))
----------------
I think this would work on APPLE (which is subsumed by UNIX). Though I don't
know if anyone there actually uses BUILD_SHARED_LIBS...
================
Comment at: lldb/source/API/CMakeLists.txt:124
+ set(_install_rpath "\$ORIGIN/../lib${LLVM_LIBDIR_SUFFIX}"
"\$ORIGIN/../../../../lib${LLVM_LIBDIR_SUFFIX}" ${extra_libdir})
+ set_target_properties(liblldb PROPERTIES INSTALL_RPATH "${_install_rpath}")
+endif()
----------------
MaskRay wrote:
> The extra_libdir logic is copied from `llvm/cmake/modules/AddLLVM.cmake`
> `llvm_setup_rpath`.
Instead of copying that you should do something like `set_property(TARGET
liblldb APPEND PROPERTY INSTALL_RPATH "whatever")` to just add the new entry.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71800/new/
https://reviews.llvm.org/D71800
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
