Bug 209538 - Allow mod_admserv to build with mozldap5/6
Summary: Allow mod_admserv to build with mozldap5/6
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: 389
Classification: Retired
Component: Admin
Version: 1.0.2
Hardware: All
OS: All
medium
medium
Target Milestone: ---
Assignee: Rich Megginson
QA Contact: Viktor Ashirov
URL:
Whiteboard:
Depends On:
Blocks: 152373 fds103trackingbug 240316
TreeView+ depends on / blocked
 
Reported: 2006-10-05 21:19 UTC by Rich Megginson
Modified: 2015-12-07 16:57 UTC (History)
0 users

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-12-07 16:57:16 UTC
Embargoed:


Attachments (Terms of Use)
diffs for fix (5.54 KB, patch)
2006-10-05 21:19 UTC, Rich Megginson
no flags Details | Diff
new diffs (5.53 KB, patch)
2006-10-05 22:52 UTC, Rich Megginson
no flags Details | Diff
cvs commit log (1.88 KB, text/plain)
2006-10-05 23:11 UTC, Rich Megginson
no flags Details

Description Rich Megginson 2006-10-05 21:19:54 UTC
mod_admserv needs to be able to build with mozldap versions 5 or 6.

Comment 1 Rich Megginson 2006-10-05 21:19:54 UTC
Created attachment 137871 [details]
diffs for fix

Comment 2 Noriko Hosoi 2006-10-05 22:26:53 UTC
It may not be necessary to think of this possibility, but if there are multiple
versions of LDAP C SDK accidentally installed in $LDAPSDKLIBDIR, e.g.,
libldap50.so, and libldap60.so, then libldap50.so would be picked up first and
it sets ldapsdk_ver=50...  I think it'd be nice if we can do it with the
descendant order (or pick up the higher number?)

+if test -z "$ldapsdk_ver" ; then
+   # figure out version of ldapsdk
+    for file in $LDAPSDKLIBDIR/libldap??.so* ; do
+        case $file in
+        *libldap60.so*) ldapsdk_ver=60 ;;
+        *) ldapsdk_ver=50 ;;
+        esac
+        break
+    done   
+fi
+

Comment 3 Rich Megginson 2006-10-05 22:52:22 UTC
Created attachment 137879 [details]
new diffs

Thanks Noriko.	Here are the new diffs.  If it finds 60 libs, it breaks out of
the for loop.  Otherwise, it keeps looking for 50 libs.

Comment 4 Noriko Hosoi 2006-10-05 23:00:36 UTC
Looks cool!

Comment 5 Rich Megginson 2006-10-05 23:11:14 UTC
Created attachment 137881 [details]
cvs commit log

Reviewed by: nhosoi (Thanks!)
Files: see diff
Branch: HEAD
Fix Description: If there is a mozldap6 package on the system, assume
the lib suffix is 60, otherwise, 50.  If no mozldap package is on the
system, look for libldap60.so* in the given ldapsdk_lib directory.  If
found, assume 60, otherwise, 50.
There were also a few problems with the way we were looking for ICU,
mostly copy/paste errors, but one was that pkg-config does not have an
icu package on some systems, so you have to use icu-config instead.
Noriko pointed out that if both 50 and 60 libs are in the same dir, the 50 libs
will be selected first - we want 60 libs to have preference. The new diffs will
select 60 libs if found, or 50 libs if not.
Platforms tested: FC5
Flag Day: no
Doc impact: no

Comment 6 Anh Nguyen 2007-12-03 15:39:26 UTC
Changed QA Whiteboard to to_be_verified_by_dev.

Comment 7 Nathan Kinder 2007-12-14 23:58:40 UTC
We are currently building adminserver (which builds mod_admserv) using mozldap
version 6 as the default across numerous platforms, even if mozldap 5 is on the
system.  Marking as verified.


Note You need to log in before you can comment on or make changes to this bug.