389-ds-base

#50332 Ticket 50302 - remove support for mozldap
Closed by spichugi. Opened by firstyear.
firstyear/389-ds-base 50302-remove-mozilla-ldapsdk  into  master

Download 50332.patch

Bug Description: Mozldap has not been supported for some time in favour
of the OpenLDAP libraries for protocol implementation. Remove the unused
mozldap

Fix Description: Remove the compat header and defined-out code, along
with the ability to configure this as the ldapsdk.

https://pagure.io/389-ds-base/issue/50302

Author: William Brown william@blackhats.net.au

Review by: ???

Nice work, @firstyear. But it looks like you forgot to remove the ldapsdk_* variables from configure.ac, Makefile.am and some other files.

Also, I wasn't sure how the src/svrcore work was related to the Mozilla LDAP SDK (if it is).
Edited by hmc

Probably, @mhonek has more to say, but what about all of the blocks (a lot of them) like: 

#if defined(USE_OPENLDAP)
#else /* !USE_OPENLDAP */

I think they should be cleaned up too if we remove the mozldap support...
Edited by spichugi

Pull-Request has been closed by firstyear

Closed in favour of #50334

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3391

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.