FindKDE4Internal.cmake: only conditionally set(CMAKE_INSTALL_RPATH_USE_LINK_PATH TRUE)?

[Rex Dieter]

Historically in our fedora (and other distros) packaging, folks just brute 
force
set(CMAKE_INSTALL_RPATH_USE_LINK_PATH FALSE)
to avoid the rpath mess in default prefixes.

But, with great hope I look forward to the day this all "just works", and we 
can avoid rpath'ing anything in 
CMAKE_PLATFORM_IMPLICIT_LINK_DIRECTORIES
without much ado.  Attached is my latest attempt to accomplish that by 
conditinally doing
set(CMAKE_INSTALL_RPATH_USE_LINK_PATH TRUE)
*only* if any non-implicit items are actually added to CMAKE_INSTALL_RPATH

Comments?

If something like this approach is acceptable, I'd like to go through all 
the kdesupport projects that copied the unconditional items
set(CMAKE_INSTALL_RPATH_USE_LINK_PATH  TRUE)
set(CMAKE_INSTALL_RPATH "${LIB_INSTALL_DIR}")
(from older versions of FindKDE4Internal.cmake) too.

-- Rex
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kdelibs-4.5.95-rpath-2.patch
Type: text/x-patch
Size: 826 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/kde-buildsystem/attachments/20110121/fa130254/attachment.patch