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

[Stephen Gallagher]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Simo Sorce wrote:
> 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.
> 
> 

Yeah, I just happened to try running it, and I saw the error there.

Ack.

- --
Stephen Gallagher
RHCE 804006346421761

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org

iEYEARECAAYFAknU3i0ACgkQeiVVYja6o6MdbwCfVc2H0coJePDdkzb/f8uTEIdD
kYkAmwdd051WKa/HjqTei8UJgTVT1Z29
=cvzA
-----END PGP SIGNATURE-----