From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40665) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dtZVP-0005Lz-0V for guix-patches@gnu.org; Sun, 17 Sep 2017 09:21:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dtZVN-0004ek-Vb for guix-patches@gnu.org; Sun, 17 Sep 2017 09:21:03 -0400 Received: from debbugs.gnu.org ([208.118.235.43]:35857) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dtZVN-0004eT-SP for guix-patches@gnu.org; Sun, 17 Sep 2017 09:21:01 -0400 Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dtZVN-0005TV-LV for guix-patches@gnu.org; Sun, 17 Sep 2017 09:21:01 -0400 Subject: [bug#28444] [PATCH 3/3] build-system: Add 'meson-build-system'. Resent-Message-ID: From: ludo@gnu.org (Ludovic =?UTF-8?Q?Court=C3=A8s?=) References: <20170913125003.13313-1-petermikkelsen10@gmail.com> <20170913125003.13313-3-petermikkelsen10@gmail.com> <87a81vllgk.fsf@gnu.org> <87ingik4hc.fsf@gnu.org> Date: Sun, 17 Sep 2017 15:19:50 +0200 In-Reply-To: (Peter Mikkelsen's message of "Sat, 16 Sep 2017 19:06:03 +0200") Message-ID: <87vakhihs9.fsf@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-patches-bounces+kyle=kyleam.com@gnu.org Sender: "Guix-patches" To: Peter Mikkelsen Cc: 28444-done@debbugs.gnu.org Hi Peter, Peter Mikkelsen skribis: > and without the shrinking done, the runpath looks like this: > > /gnu/store/8y6wd2rfi6p3gpxcz3p1gyzsbxwyc9ha-uuid-cpp-0.6.5.1/lib:$ORIGIN/= :/gnu/store/n6nvxlk2j8ysffjh3jphn1k5silnakh6-glibc-2.25/lib:/gnu/store/3x53= yv4v144c9xp02rs64z7j597kkqax-gcc-5.4.0-lib/lib:/gnu/store/3x53yv4v144c9xp02= rs64z7j597kkqax-gcc-5.4.0-lib/lib/gcc/x86_64-unknown-linux-gnu/5.4.0/../../= .. > > and with, it looks like this: > > /gnu/store/nqdf6kr5cqfqh9z5yprar1yyfqwprj4v-uuid-cpp-0.6.5.1/lib:$ORIGIN/= :/gnu/store/n6nvxlk2j8ysffjh3jphn1k5silnakh6-glibc-2.25/lib:/gnu/store/3x53= yv4v144c9xp02rs64z7j597kkqax-gcc-5.4.0-lib/lib > > So the last part of the old runpath is removed, and looking at the > output of ldd, it seems like it was never needed: > > linux-vdso.so.1 (0x00007ffd6cbe2000) > libuuid.so =3D> /gnu/store/8y6wd2rfi6p3gpxcz3p1gyzsbxwyc9ha-uuid-cpp-0.6.= 5.1/lib/libuuid.so > (0x00007feae61bc000) > libstdc++.so.6 =3D> > /gnu/store/3x53yv4v144c9xp02rs64z7j597kkqax-gcc-5.4.0-lib/lib/libstdc++.s= o.6 > (0x00007feae5e42000) > libgcc_s.so.1 =3D> > /gnu/store/3x53yv4v144c9xp02rs64z7j597kkqax-gcc-5.4.0-lib/lib/libgcc_s.so= .1 > (0x00007feae5c2b000) > libc.so.6 =3D> /gnu/store/n6nvxlk2j8ysffjh3jphn1k5silnakh6-glibc-2.25/lib= /libc.so.6 > (0x00007feae588c000) > libm.so.6 =3D> /gnu/store/n6nvxlk2j8ysffjh3jphn1k5silnakh6-glibc-2.25/lib= /libm.so.6 > (0x00007feae557a000) > /gnu/store/n6nvxlk2j8ysffjh3jphn1k5silnakh6-glibc-2.25/lib/ld-linux-x86-6= 4.so.2 > (0x00007feae63c4000) > > Now, I am not sure it is worth shrinking, but I just think it is a > little bit strange having unneeded directories in the runpath :) The last element of the RUNPATH above comes from GCC or so. I can=E2=80=99t hurt and you=E2=80=99ll probably find it on all the binaries we build; it= =E2=80=99s not related to Meson, I think. So if that=E2=80=99s the only reason for =E2=80=98patchelf --shrink-runpath= =E2=80=99, I would argue it=E2=80=99s not a valid one. :-) > While trying to build a gnome package, I noticed that the runpath was > huuuge before shrinking, but I don't have that package definition > anymore :/ Unless you find a more compelling example ;-) as you build packages with =E2=80=98meson-build-system=E2=80=99, I=E2=80=99d suggest we eventually rev= isit this issue and get rid of =E2=80=98fix-runpath=E2=80=99. That would remove the dependency= on PatchELF, which is sometimes a bit fragile. How does that sound? Thanks, Ludo=E2=80=99.