[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#28444] [PATCH 3/3] build-system: Add 'meson-build-system'.

From: Peter Mikkelsen
Subject: [bug#28444] [PATCH 3/3] build-system: Add 'meson-build-system'.
Date: Sat, 16 Sep 2017 19:06:03 +0200

2017-09-16 18:11 GMT+02:00 Ludovic Courtès <address@hidden>:
> Hi Peter,
> Peter Mikkelsen <address@hidden> skribis:
> [...]
> >> According to this description, half of it corresponds to the
> >> ‘validate-runpath’ phase, no?
> >
> > To my understanding 'validate-runpath' just checks and complains, but
> > this phase checks if it can find any dependencies in directories that
> > are local to the package, and if it can, it adds them to the runpath.
> > I have tried to update the description :)
> I see.  Out of curiosity, which libraries did you find to be present in
> RUNPATH but not in NEEDED?

I tried building a simple program, and using 'patchelf --print-needed'
i got this:

and without the shrinking done, the runpath looks like this:


and with, it looks like this:


So the last part of the old runpath is removed, and looking at the
output of ldd, it seems like it was never needed: (0x00007ffd6cbe2000) => 
(0x00007feae61bc000) =>
(0x00007feae5e42000) =>
(0x00007feae5c2b000) => 
(0x00007feae588c000) => 

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 :)
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 :/
We can see what happens when we update gnome :)
> I’d consider it a Meson bug (or a bug in how the package uses Meson) if
> test libraries are passed as -l when linking non-test binaries.

I don't know if this is what is happening, but on their side they
'fix' the runpath before install, which we don't.
If any problem arises I will be happy to give it a look, but in the
mean time, I think this will work :)

> At any rate, we should implement ‘shrink-runpath’ in (guix build
> gremlin) someday!

That would be awesome!
> > From 8185c2a154c7473d3b50351246cc83b792ec6a57 Mon Sep 17 00:00:00 2001
> > From: Peter Mikkelsen <address@hidden>
> > Date: Wed, 13 Sep 2017 14:37:39 +0200
> > Subject: [PATCH] build-system: Add 'meson-build-system'.
> >
> > * (MODULES): Add 'guix/build-system/meson.scm' and
> >   'guix/build/meson-build-system.scm'.
> > * guix/build-system/meson.scm: New file.
> > * guix/build/meson-build-system.scm: New file.
> > * doc/guix.texi (Build Systems): Add 'meson-build-system'.
> Pushed with the changes below.
> Thank you!
> Now to upgrade GNOME?  :-)
Yes :) I will finish some ongoing Haskell work before giving it a shot
> Ludo’.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]