[pacman-dev] [PATCH 3/4] Build vercmp without needing link to libalpm
dpmcgee at gmail.com
Wed May 5 14:55:25 CEST 2010
On Wed, May 5, 2010 at 1:05 AM, Allan McRae <allan at archlinux.org> wrote:
> On 05/05/10 14:45, Dan McGee wrote:
>> Include the object file directly from the libalpm version comparison code
>> it is the only thing we need. This drops the dependency of vercmp on
>> libalpm and all of the stuff we know it drags in.
>> Signed-off-by: Dan McGee<dan at archlinux.org>
> After applying the four patches from this patchset:
>> readelf -d src/util/vercmp
> Dynamic section at offset 0xdfc contains 21 entries:
> Tag Type Name/Value
> 0x00000001 (NEEDED) Shared library: [libssp.so.0]
> 0x00000001 (NEEDED) Shared library: [libc.so.6]
> libssp is part of gcc(-libs), so linking to that is probably not a disaster.
> But is it still not needed for the current Arch vercmp so may be worth
> looking into.
The ssp comes from here:
See the GCC_STACK_PROTECT_* macros listed there, which are turned on
in debug builds (and for now not in production builds, although there
is a feature request in Arch in general to add that).
> This was of course using -Wl,--as-needed. The list of linked libraries is
> very long if that is not used (I discovered I had an unaltered
> makepkg-git.conf...). Maybe we should consider adding --as-needed by
> default (when supported). Almost ever system that can build pacman will
> support --as-needed.
Yeah, sorry I sent these late or I would have noted you needed to use
--as-needed. I think for now that is a reasonable compromise, and you
might be right about saying we could add that as default.
Thanks for testing!
More information about the pacman-dev