[Fedora-directory-devel] Please review: [2] Bug 209538: Allow mod_admserv to build with mozldap5/6
by Rich Megginson
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209538
Bug(s) fixed: 209538
Bug Description: Allow mod_admserv to build with mozldap5/6
Reviewed by: ???
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
QA impact: should be covered by regular nightly and manual testing
New Tests integrated into TET: none
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=137879
17 years, 6 months
[Fedora-directory-devel] Please review:Bug 209538: Allow mod_admserv to build with mozldap5/6
by Rich Megginson
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209538
Bug(s) fixed: 209538
Bug Description: Allow mod_admserv to build with mozldap5/6
Reviewed by: ???
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.
Platforms tested: FC5
Flag Day: no
Doc impact: no
QA impact: should be covered by regular nightly and manual testing
New Tests integrated into TET: none
https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=137871
17 years, 6 months
[Fedora-directory-devel] Please review (revised): [Bug 208672] parameterizing the hardcoded paths (phase 2. db, log, lock, pid dir)
by Noriko Hosoi
Hello,
After discussing with Nathan, I've removed nsslapd-dbdir and
nsslapd-ldifdir from cn=config, which are redundant.
Your review would be greatly appreciated.
Thanks,
--noriko
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=208672
Summary: parameterizing the hardcoded paths (phase 2. db, log,
lock, pid dir)
Product: Fedora Directory Server
Version: 1.0.2
Platform: All
OS/Version: Linux
Status: NEW
Severity: normal
Priority: normal
Component: Directory Server
AssignedTo: nhosoi(a)redhat.com
ReportedBy: nhosoi(a)redhat.com
QAContact: ohegarty(a)redhat.com
CC: ohegarty(a)redhat.com
------- Additional Comments From nhosoi(a)redhat.com 2006-10-03 19:45 EST -------
Created an attachment (id=137712)
--> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=137712&action=view)
cvs diffs (ldapserver -- revised)
Thanks for the comments and the discussion, Nathan.
Here's the additional changes:
1) I re-thought about the 2 issues and have decided to get rid of nsslapd-dbdir
and nsslapd-ldifdir from cn=config. Instead of referring nsslapd-dbdir, we can
use nsslapd-directory in cn=config,cn=ldbm database,cn=plugins,cn=config. And
for nsslapd-ldifdir, it's basically "defined" in db2ldif, where you can get the
location info.
2) In terms of db2ldif and db2ldif.pl, I added server_id prefix to the default
exported file name. Before, each instance had its own ldif dir, but under HFS,
there is one ldif directory shared among servers on one host. To make it
easier to know the origin of the ldif, I propose to add <server_id>- in front
of the generated ldif file name like this:
<prefix>/share/fedora-ds/ldif/<serverID>-2006_10_03_162640.ldif
3) In the same token, when another instance is added to an existing server
sharing the same directory structure, default ldif files such as Example.ldif
and Example-roles.ldif files are overwritten. So, I added a backup code to
keep the existing files. (note that if the file having the extension ".bak"
already exists, the file is overridden...)
Files:
ldap/admin/include/dsalib.h
ldap/admin/lib/dsalib_db.c
ldap/admin/lib/dsalib_location.c
ldap/admin/lib/dsalib_util.c
ldap/admin/src/create_instance.c
ldap/admin/src/create_instance.h
ldap/admin/src/ds_remove.c
ldap/admin/src/scripts/template-db2ldif.pl
ldap/servers/slapd/libglobs.c
ldap/servers/slapd/protect_db.c
ldap/servers/slapd/proto-slap.h
ldap/servers/slapd/slap.h
ldap/servers/slapd/snmp_collator.c
ldap/servers/slapd/back-ldbm/ldbm_config.c
ldap/servers/slapd/back-ldbm/ldbm_config.h
17 years, 6 months