Jyri Virkki wrote: > Jeff Trawick wrote: > >> The native Solaris LDAP and OpenLDAP SDKs implement some of the >> same symbols. Any libraries loaded into the same process as >> APR-Util and Apache must use the same LDAP SDK, or the behavior >> is undefined. >> >> There are believed to be no other users of the APR-Util LDAP >> interfaces besides Apache, but it is is unknown if there are >> other users of APR-Util in general which separately use LDAP. >> The LDAP SDK use in APR-Util is segregated to the separate >> library apr_ldap.so which is loaded dynamically if APR-Util >> LDAP functions are called, so it shouldn't be a problem for >> such APR-Util applications which separately use LDAP. >> > > Since this is the potentially controversial part of the proposal, the > spec should preemptively answer questions about it. > > "unknown" and "shouldn't" are bad words in an ARC case... > > I recommend checking a recent nevada build to list everything which > links with APR and confirming those app(s) don't have any issues with > the change and documenting that investigation. For approval, the case > needs to convincingly show this change won't cause breakage elsewhere > in nevada/OpenSolaris. > > That was a global statement for all platforms, which is necessarily imprecise ("There are believed to be no other users of the APR-Util LDAP interfaces besides Apache, but it is is unknown if there are other users of APR-Util in general which separately use LDAP.")
As far as references to libaprutil in Nevada (b105), there are (after running ldd on all executable files under /usr and /opt) 1. Apache and its commands in SUNWapch22 these currently reference Solaris libldap because Apache uses it, but that will be changed by this project 2. mod_perl's APR.so (no reference to APR-Util ldap functions), also packaged within SUNWapch22 not impacted since it doesn't use the native LDAP SDK it references Solaris libldap, but ldd -u reports it as unused 3. many Subversion commands and libraries in SUNWsvn all of these also reference Solaris libldap, but ldd -u reports it as unused 4. Subversion Perl interface in SUNWsvn-perl all of these also reference Solaris libldap, but ldd -u reports it as unused 5. Subversion Python interface in SUNWsvn-python all of these also reference Solaris libldap, but ldd -u reports it as unused -/- Note that *anything* that references libapr picks up an unused reference to libldap. >> 4.5. Interfaces: >> >> This affects both APR-Util and Apache. >> >> Interfaces removed: >> > > To avoid confusion I'd write > "Imported Interfaces removed (no longer imported)" > > >> NAME STABILITY NOTES >> >> LDAP Evolving PSARC/1997/276 et seq. >> >> Interfaces added: >> > > And same here "Imported Interfaces added:" > > >> NAME STABILITY NOTES >> >> OpenLDAP External/Volatile PSARC/2008/507 >> > > "External" is an old classification no longer used. In fact, reading > 2008/507 I see it doesn't declare anything to be "External". So it is > "Volatile", above. > > > > >> 4.6.1. Implications for Apache configuration >> >> Refer users to the OpenLDAP-specific details for configuring ldaps >> connections at >> >> http://httpd.apache.org/docs/2.2/mod/mod_ldap.html#settingcerts >> >> A sample LDAP configuration showing ldaps connections to the Sun >> Directory Server is recommended. >> > > Nit, but recommended to whom? > I guess above is saying next apache integration will include such a > sample? If so, state it that way: "A sample LDAP configuration > showing ldaps connections to the Sun Directory Server will be included." > > > That's a documentation section (" 4.6. Doc Impact:"), so it is a recommendation to the doc team to show an example configuration. -/- Any additional/continuing concerns, or should I try to clarify these issues in the ARC draft? Thanks as always, Jeff