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

[pkg-wine-party] Move sonames2elf to dpkg-shlibdeps? (was: Move sonames2elf to debhelper?)

On 02/05/2017 09:24 PM, Michael Gilbert wrote:
> On Sun, Feb 5, 2017 at 3:06 PM, Jens Reyer wrote:
>> On 02/05/2017 08:34 PM, Michael Gilbert wrote:
>>> sonames2elf seems like it solves a broader problem and probably
>>> doesn't really belong in the wine package itself.
>>> Would it be possible to add it as an option to dh_shlibdeps?  That
>>> might also help solve some older debhelper bugs like #35733?
>> Yes, definitely.  It has already been suggested very long ago
>> (#596715, dpkg-shlibdeps: Please allow to manually add
>> library dependencies via shlibdeps).
>> Unfortunately (without having looked at the debhelper code) I assume
>> that is beyond my capabilities.
> dh_shlibdeps is only about 100 lines of perl. It should be somewhat
> straightforward to implement sonames2elf using perl's system call to
> call cc.
> dpkg-shlibdeps is also only about 500 lines of perl, if that's where it belongs.

Sorry for my previous sketchy answer mixing up debhelper and
dpkg-shlibdeps: according to https://bugs.debian.org/596715#10
dpkg-shlibdeps is the right place to implement this.

> Good luck!

Thanks!  I had a look at it, but unfortunately I don't think I'm able to
solve this in the foreseeable future - I'm just too unfamiliar with perl
(as basically any language that's not shell), and I don't have enough
time for changing this currently.

So I'd be grateful to anybody who tackles this.  At DebConf16 Nils
Thykier showed a little bit of interest in looking into this, but no
promises were made at all.

Thus having said, while directly handling this with dpkg-shlibdeps is
the "perfect" solution, imo the current implementation is quite "good"
and does its job.  Besides the recent issue caused by the use of
"-nostdlib" (which had also caused build failures on Ubuntu arm
architectures), I'm only aware of one issue in Ubuntu where
_unversioned_ sonames are necessary so that libGL.so is found.


Reply to: