Hi, On Tue, Dec 26, 2017 at 12:55:40PM +0100, Michael Weiser wrote:
> > I think the cmake patches are ok, I'll look into the ebuilds you > > mention. I'll sync whatever I can find in this area. > Actually, the cmake patch might be incorrect since it got lost on emaint > sync and I had to reconstruct it from memory. I'm sure about > CMAKE_BUILD_WITH_INSTALL_RPATH, CMAKE_INSTALL_RPATH, > CMAKE_INSTALL_RPATH_USE_LINK_PATH but I think I also dropped > CMAKE_INSTALL_NAME_DIR for llvm and clang. After some more playing with cmake I think I got it: We do want a minimal CMAKE_INSTALL_RPATH (with just the path this project's libraries get installed to) and CMAKE_INSTALL_RPATH_USE_LINK_PATH because of https://cmake.org/Wiki/CMake_RPATH_handling#Always_full_RPATH. This way cmake will automatically add the directories of libraries that a project links against and that are outside the build directory. No idea how the first part can be done without help by the upstream source or the individual ebuilds though. We do most likely not want CMAKE_BUILD_WITH_INSTALL_RPATH and CMAKE_INSTALL_NAME_DIR because we now know it causes breakage and the benefit is still unclear. -- bye, Micha
