Skip to content

Commit

Permalink
update
Browse files Browse the repository at this point in the history
git-svn-id: svn://anonsvn.mit.edu/krb5/branches/ldap-integ@17891 dc483132-0cff-0310-8789-dd5450dbe970
  • Loading branch information
raeburn committed Apr 11, 2006
1 parent e2dacff commit 76e4ac1
Showing 1 changed file with 14 additions and 8 deletions.
22 changes: 14 additions & 8 deletions blocking-bugs
Original file line number Diff line number Diff line change
Expand Up @@ -3,24 +3,30 @@
This is a list of blocking bugs that must be addressed before these
changes are committed to the trunk:

* ?
* Per discussion with Sam: Merge the not-actually-LDAP bits to the
trunk, keep working a bit on the LDAP bits.

* Move LDAP bits (library, ldap-util) into plugin tree, to collect
them all in one place.

* Re-evaluate split between plugin and library; if anything is used
only by the plugin, move it there.

* Remove mutex hacks and assume that the LDAP code either is already
thread-safe, or will be made so when we also get patches to make the
KDC multi-threaded.

Probably-non-blocking issues with this code:

* (non-blocking) Do we want --with-ldap or --enable-ldap? Should
eDirectory be a separate option?
* (non-blocking) Do we want --with-ldap or --enable-ldap? Likewise
for edirectory. Leaning towards --with.

* The export list needs to match the provided symbols. Problem is,
the openldap and edirectory cases provide two different sets of
symbols. For now, I've set the list to that used in the openldap
case. Longer term, perhaps stubs should be provided for the extra
names needed in the edirectory case.

* (non-blocking) It appears that the eDirectory code as submitted --
well, as Ken got it from Sam -- may not have been tested. Case in
point: It looks like it'll add "HAVE_EDIRECTORY" (*without* "-D" in
front) to the compilation flags in a couple of directories.

* Various symbols in the kdb_ldap code duplicate symbols in the kdb5
library. The kdb_ldap library needs to link against the kdb5
library because it uses other symbols from there. It's not clear to
Expand Down

0 comments on commit 76e4ac1

Please sign in to comment.