Created attachment 407628 [details, diff] cmake-utils.eclass-rpath.patch Hello, cmake allows to disable RPATH linking (enabled by default), this leads to insecure rpath warning during install phase checks, logs show that build directory (-Wl,--rpath /var/tmp/portage/...) is added to linker options. Many ebuilds fix this by adding one of the following or similar options: -DCMAKE_SKIP_RPATH=ON -DENABLE_RPATH=OFF -DCMAKE_BUILD_WITH_INSTALL_RPATH=OFF I propose to add this functionality to the eclass, thus ebuilds can be simplified. See attached patch.
It looks good to me but i would prefer if you send the patch to -dev ml to get a wider review.
Done (mail is sent to dev ml).
I responded on the mailing list why I disagree with the patch.
(In reply to Michael Palimaka (kensington) from comment #3) > I responded on the mailing list why I disagree with the patch. For the record, my response was as follows: This is usually a workaround for an underlying bug. Plus, this will break any package that legitimately uses rpath. As there were no further responses/discussion, I don't think we can proceed.