From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH] gnu: kde: Add kdelibs. Date: Sun, 04 Jan 2015 22:29:59 +0100 Message-ID: <87ppau1bfs.fsf@gnu.org> References: <641d5f5d.31e3.14aaea456bc.Coremail.iyzsong@163.com> <87ppaws52q.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:41150) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Y7skM-0003mj-Q4 for guix-devel@gnu.org; Sun, 04 Jan 2015 16:30:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Y7skI-00023A-VF for guix-devel@gnu.org; Sun, 04 Jan 2015 16:30:02 -0500 Received: from hera.aquilenet.fr ([2a01:474::1]:54890) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Y7skI-00022x-OJ for guix-devel@gnu.org; Sun, 04 Jan 2015 16:29:58 -0500 In-Reply-To: <87ppaws52q.fsf@gmail.com> (=?utf-8?B?IuWui+aWh+atpiIncw==?= message of "Sat, 03 Jan 2015 15:20:45 +0800") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: =?utf-8?B?5a6L5paH5q2m?= Cc: Guix-devel =E5=AE=8B=E6=96=87=E6=AD=A6 skribis: >> Andreas Enge skribis: >> >>> RUNPATH /gnu/store/7c30kyzagk84inhnb2nbxcmnh76xgh6c-ktou= ch-4.14.2/lib:/gnu/store/4k20pkxgvfc22wpcvh6xr26ma4b619ad-glibc-2.20/lib:/g= nu/store/k7w7wdpvaqlkwq0ijz9149pqn3lq8a5n-gcc-4.8.3-lib/lib64:/gnu/store/k7= w7wdpvaqlkwq0ijz9149pqn3lq8a5n-gcc-4.8.3-lib/lib:/gnu/store/k7w7wdpvaqlkwq0= ijz9149pqn3lq8a5n-gcc-4.8.3-lib/lib/gcc/x86_64-unknown-linux-gnu/4.8.3/../.= ./.. >>> >>> (Thanks to Ludovic for the suggestion to use objdump!) >>> >>> The linker line looks like this: >>> Linking CXX executable ktouch >>> cd /tmp/nix-build-ktouch-4.14.2.drv-0/build/src && /gnu/store/sr9lv3vhv= iy151zv315zdgi2rvq97a9r-cmake-2.8.12/bin/cmake -E cmake_link_script CMakeFi= les/ktouch.dir/link.txt --verbose=3D1 >>> /gnu/store/13ahaqlmniysmbls5d1fcmzy59yd5xk5-gcc-4.8.3/bin/c++=20 >> >> [...] >> >>> -o ktouch -rdynamic /gnu/store/mipmjv4mpsf3rrsk1mbhpqzlw939v6ny-kdelibs= -4.14.2/lib/libkdeclarative.so.5.14.2 /gnu/store/mipmjv4mpsf3rrsk1mbhpqzlw9= 39v6ny-kdelibs-4.14.2/lib/libplasma.so.3.0.0=20 >> >> [...] >> >>> You see lots of "...so" and no "-l...", which is apparently the cmake w= ay of >>> doing things. >> >> Our ld-wrapper is ineffective when the .so file names are passed >> directly like this. That is, ld-wrapper adds -rpath for any -l switch, >> but it does not add -rpath for libraries whose absolute file name is >> specified. >> >> That could be easily fixed, but in core-updates. >> >>> But no "-Wl,-rpath" despite >>> -DCMAKE_INSTALL_RPATH_USE_LINK_PATH=3DTRUE >>> Does this mean that this option only honours paths coming from "-l", wh= ile >>> at the same time using "-l" is unusual with cmake? >> >> Uh. While we can improve ld-wrapper, the best would of course be to get >> CMake to do the right thing... > We have not change ld-wrapper for this right? I=E2=80=99ve made the change in d8491ba (core-updates), which Hydra is now building. Please report any issues. Ludo=E2=80=99.