[Pkg-samba-maint] Bug#833287: Bug#833287: feedback request about patch

[Mathieu Parent]

2017-06-14 9:37 GMT+02:00 Mathieu Parent <math.parent at gmail.com>:
> 2017-06-13 17:12 GMT+02:00 Andreas Hasenack <andreas at canonical.com>:
>> Hi,
>>
>> this bug was also filed against Ubuntu as
>> https://bugs.launchpad.net/ubuntu/+source/samba/+bug/1584485, and in
>> https://bugs.launchpad.net/ubuntu/+source/samba/+bug/1584485/comments/17 a
>> patch was proposed to build libnss-winbind and libpam-winbind statically
>> against the samba-libs.
>>
>> That introduced a regression (https://bugs.launchpad.net/bugs/1677329),
>> however: pam_winbind had unresolved symbols at runtime and couldn't be
>> loaded. The patch was reverted, and by mistake added back again at some
>> later point.
>>
>> I'm linking a new version of that patch that fixes the regression, but I'm
>> not familiar with the samba4 buildsystem and would like some feedback. I'm
>> still working on the DEP3 header, the history of this one is a bit
>> complicated.
>
> Another way to fix this is to tune the dependencies (using pre-depends
> or Conflicts).
>
> Can you provide more info of which upgrade broke? (source/dest version)

More info:

In Debian, we have this:

samba (2:4.4.5+dfsg-3) unstable; urgency=medium

  [ Jelmer Vernooij ]
  * Add strict dependencies on samba-libs, because of use of private
    libraries without stable ABI across Samba binary packages.
  * Add Breaks clauses for older versions of samba-libs and samba to
    samba-vfs-modules, as some files have moved. Closes: #833164,
    #832880
[...]
  [ Mathieu Parent ]
[...]
  * Drop dependency on samba-libs in libwbclient0 to avoid
    circular dependency.


With those commits:
https://anonscm.debian.org/cgit/pkg-samba/samba.git/commit/debian/control?id=025c8cb2a00ee534bae8613e936933694d359f32
https://anonscm.debian.org/cgit/pkg-samba/samba.git/commit/debian/control?id=ea53efa334a8a0b86a611a70ac058e2a79fdc1c7
https://anonscm.debian.org/cgit/pkg-samba/samba.git/commit/debian/control?id=a4634d40a3d754921b4fe42154ae14b19bd22c5f

I'm unable to reproduce this with a jessie to stretch upgrade (yet).

-- 
Mathieu Parent