Proposal to remove zlib from the Samba tree.
ira at samba.org
Wed Jul 16 17:09:35 MDT 2014
On Wed, Jul 16, 2014 at 7:08 PM, Andrew Bartlett <abartlet at samba.org> wrote:
> On Sun, 2014-06-29 at 17:14 +0200, Julien Kerihuel wrote:
> > On 29/06/14 10:19, Andrew Bartlett wrote:
> > > On Sat, 2014-06-28 at 08:37 +0100, jelmer at samba.org wrote:
> > >> I like the idea of removing it and always using the system zlib.
> > >>
> > >> Please make sure that the build farm hosts all have zlib installed in
> a way that Samba can use it.
> > > It would be very useful to know what systems don't have (and can't
> > > reasonably get) the needed version of zlib on them, to understand the
> > > best approach here.
> > >
> > > While zlib is 'only' needed for NDR_COMPRESSION, that is in turn used
> > > for DRSUAPI, which is used in the source3 client as part of net rpc
> > > samsync, so it isn't easily made an optional dependency.
> > It is also intensively used in openchange server and mapiproxy to
> > decompress MAPI buffers in Outlook client requests. If the dependency is
> > made optional, then we are going to make the life of users building the
> > samba+openchange stack from sources more difficult while we are aiming
> > at smooth, nice and out of the box integration with samba.
> Is that used via Samba, or directly? I wasn't able to see use of the
> NDR compression routines in openchange.
> It still is a public ABI that we would be breaking, so we would have to
> also not install libndr (because some other package might use it), but
> it would be very helpful for you to confirm if this would specifically
> break openchange.
I have pulled this change off the table. I think my most recent work
should work for everyone.
More information about the samba-technical