[Freeipa-devel] [PATCH] sssd: change from reserved ldap_ to sdap_ in ldap provider

Simo Sorce ssorce at redhat.com
Thu Apr 2 15:46:17 UTC 2009


On Thu, 2009-04-02 at 11:26 -0400, Stephen Gallagher wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Simo Sorce wrote:
> > Better not to use the ldap_ name space at all for our internal function
> > names. Let it be reserved for ldap libraries.
> > 
> > Simo.
> > 
> > 
> > 
> > ------------------------------------------------------------------------
> > 
> > _______________________________________________
> > Freeipa-devel mailing list
> > Freeipa-devel at redhat.com
> > https://www.redhat.com/mailman/listinfo/freeipa-devel
> 
> Nack:
> 
> Line 218: This needs to be left alone as ldap_start_tls(), it's an
> actual LDAP call. Please also revert the DEBUG statements related to it.

the funny thing is that it actually compiled ...

new patch attached

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Do-not-use-the-ldap-libraries-ldap_-prefix.patch
Type: text/x-patch
Size: 12536 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20090402/5f48c34b/attachment.bin>


More information about the Freeipa-devel mailing list