Pick up latest LDAP c-sdk fixes

RESOLVED FIXED in Thunderbird 3.0rc1

Status

MailNews Core
Build Config
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

Trunk
Thunderbird 3.0rc1
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
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.
(Assignee)

Comment 1

8 years ago
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.
(Assignee)

Comment 2

8 years ago
(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.
(Assignee)

Comment 3

8 years ago
Created attachment 404904 [details] [diff] [review]
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.
Attachment #404904 - Flags: superreview+
Attachment #404904 - Flags: review+
Attachment #404904 - Flags: approval-thunderbird3+
(Assignee)

Comment 4

8 years ago
Checked in: http://hg.mozilla.org/comm-central/rev/8986804c1bb1
(Assignee)

Comment 5

8 years ago
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.
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.