LDAP functions in 4.0

[David Faure]

On Thursday 13 July 2006 11:11, Stephan Kulow wrote:
> Am Donnerstag, 13. Juli 2006 10:34 schrieb Szombathelyi György:
> > Tobias Koenig írta:
> > > On Wed, Jul 12, 2006 at 11:35:04PM +0200, Szombathelyi György wrote:
> > >> Hello!
> > >
> > > Hi György,
> > >
> > >> Also my plan is to take
> > >> some classes from KAddressBook which are LDAP related (LdapServer and
> > >> some others), clean up for generic use and also put it in this lib. My
> > >> final goal is to make a lib which can be used for basic LDAP
> > >> manipulation, and querying directory servers will not require kio_ldap
> > >> (as KUser, KAddressbook and the LDAP resource does it now), actually
> > >> kio_ldap will be a wrapper for this lib. Opinions?
> > >
> > > Great, I'm all for it!
> > >
> > > We just have to think about where to locate this lib in SVN, if it's a
> > > generic lib, most applications should be able to make use of it.
> >
> > Put it to a seperate module in kdesupport? That has the drawback that
> > another module needed to compile in some KDE functions.
> 
> Put the library in kdepimlibs - kdebase will require kdepimlibs pretty soon 
> anyway I'm sure. It just doesn't make sense otherwise ;(

I disagree, wouldn't it be simpler if kdebase did not require kdepimlibs?
It doesn't really need it, except for kio_ldap. I suggested on kdepim@ to move 
kio_ldap to kdepim, but the answer was "but coolo said ok for this dependency
on kde-core-devel" so now I'm replying here ;)

kio_ldap really belongs in kdepim imho. One proof is that in proko2 we had to move
it there anyway, to modify it ;)

But Szombathelyi György says maybe kdm wants to use kldap at some point...
Is that true, at least conceptually?

(He also says kdeadmin/kuser uses kio_ldap, but I guess it would make more sense to
port that to the kldap library instead anyway, no?). If that kldap library is so much used
everywhere then maybe it's more something for kdelibs than for kdepimlibs...

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).