520476 - Pick up latest LDAP c-sdk fixes

Status: RESOLVED FIXED

(MailNews Core :: Build Config, defect)

Description

[Mark Banner (:standard8)]

Bug 520401 (Work around Mac ppc/gcc 4.2 compiler hang in ufn.c) means comm-central is going to need to pick up a newer LDAP c-sdk release.

At the same time, as long as there are no big changes I think we may as well pick up the latest LDAP c-sdk for Thunderbird 3.

Comment 1

[Mark Banner (:standard8)]

Current changes in LDAP c-sdk not in comm-central version:

* added support for MozillaBuild using msys (no bug)
* Bug 485690: LDAP SSL connections with clientauth sometimes use wrong credentials
* Bug 436981: LDAP c-sdk: configure detects x86_64 linux as x86.
* Bug 500001: fix compiler warnings in c-sdk/ldap
** This is mainly just fixes to debug printfs etc or NULL -> 0 or unused variable warnings.
* Bug 511817 Malformed Packet appears in network trace of SASL binding.

Comment 2

[Mark Banner (:standard8)]

(In reply to comment #1)
> Current changes in LDAP c-sdk not in comm-central version:
> 
> * added support for MozillaBuild using msys (no bug)

Unfortunately our try server showed that this change is suspect. I therefore backed it out before I tagged the LDAP c-sdk.

I will file a follow up bug later.

These bugs will be incorporated:

> * Bug 485690: LDAP SSL connections with clientauth sometimes use wrong
> credentials
> * Bug 436981: LDAP c-sdk: configure detects x86_64 linux as x86.
> * Bug 500001: fix compiler warnings in c-sdk/ldap
> ** This is mainly just fixes to debug printfs etc or NULL -> 0 or unused
> variable warnings.
> * Bug 511817 Malformed Packet appears in network trace of SASL binding.

as will

* Bug 520401 Work around Mac ppc/gcc 4.2 compiler hang in ufn.c

The tag is LDAPCSDK_6_0_6C_MOZILLA_RTM.

Comment 3

[Mark Banner (:standard8)]

Attachment: Update LDAP c-sdk tag

Patch which updates the tag. All the changes have been reviewed by the LDAP C-sdk team, therefore r/sr=me for the upgrade.

a=me as there isn't anything large in the changes that could cause significant regressions, and I believe we've got enough ldap users that we would find issues before final.

Comment 4

[Mark Banner (:standard8)]

Checked in: http://hg.mozilla.org/comm-central/rev/8986804c1bb1

Comment 5

[Mark Banner (:standard8)]

I ended up having to take a bustage fix for some incorrect non-c-style comments. So we ended up with the tag: LDAPCSDK_6_0_6D_MOZILLA_RTM

http://hg.mozilla.org/comm-central/rev/e2154e9d6b01

This bug is now fixed. I'm currently doing a try server build with LDAP c-sdk HEAD to see about the issue with "added support for MozillaBuild using msys", if that fails again I'll raise an appropriate bug.