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

Re: RFS: libgeier (updated package)



Andreas Moog <amoog@ubuntu.com> writes:

> On 07/11/2011 10:03 PM, Olaf Dietsche wrote:
>> Andreas Moog<amoog@ubuntu.com>  writes:
>>
>>> On 07/11/2011 08:17 PM, Olaf Dietsche wrote:
>>>
>>>>> I didn't look into your package, but still having an empty
>>>>> dependency_libs entry is important for multiarch support. According to
>>>>> http://release.debian.org/~aba/la/current.txt (which is linked from
>>>>> http://lists.debian.org/debian-devel/2011/05/msg01003.html) xmlsec1 is
>>>>> OK, so you should really clean this entry in your libgeier.la.
>>>>
>>>> I don't understand this either.
>>>>
>>>> If xmlsec1 is ok and libtool copies these dependencies into libgeier.la,
>>>> why is it an error for libgeier to include this information, but not for
>>>> xmlsec1?
>>>
>>> If you look at libxmlsec1-dev you will notice that the dependencies_libs
>>> entry is empty, as it should be.
>>
>> $ dpkg -l 'libxmlsec1*'
> [...]
>> ii  libxmlsec1        1.2.14-1+squeeze1 XML security library
>
> Squeeze? The current version in unstable is 1.2.14-1.2.

Well, my current version is squeeze :-)

Anyway, I looked into unstable and patched /usr/lib/libxmlsec1*.la
temporarily, which means clearing dependency_libs.

Unfortunately, this doesn't help. Libtool (ld) still links libgeier
against all the (unnecessary) libs as before. dpkg-shlibdeps warns about
and ldd libgeier.so outputs the same list as before.

So, clearing dependency_libs alone, doesn't help really, because the
additional libs are also introduced through pkg-config. Maybe it would
help, if libtool added something like "--as-needed" or similar to the
link command line. But I'm neither a libtool nor a ld expert.

>>>> If this is a bug for libgeier, libtool shouldn't copy these
>>>> dependencies in the first place.
>>>>
>>>> And if an empty dependency_libs entry is important for multiarch
>>>> support, why is it important for libgeier, but not for xmlsec1?
>>>
>>> It's important for ALL packages to either ship no la-file at all (if no
>>> other package references it) OR to have at least an empty
>>> dependencies_libs entry in the la-file. libxmlsec1-dev's la-file is
>>> empty, libgeier's isn't.
>>
>> If libxmlsec1-dev's were empty, we wouldn't discuss this matter for
>> libgeier.
>
> See above, in the current unstable version the dependency_libs field
> is empty.

Is there any dh_fixup_la_file debhelper or is it up to every single
maintainer to add some sed fiddling?

Regards, Olaf


Reply to: