[389-devel] Please review: OpenLDAP support
Rich Megginson
rmeggins at redhat.com
Tue Jul 7 14:34:31 UTC 2009
Nathan Kinder wrote:
> On 07/06/2009 12:20 PM, Rich Megginson wrote:
>> Note - the patch does not contain the diffs for configure nor
>> Makefile.in
>> http://rmeggins.fedorapeople.org/0001-OpenLDAP-support.patch
> There are a few minor log message formatting issues that I noticed:
>
> - Do a grep for "insee_if_write_available" for the first one. A space
> just needs to be added between "in" and "see_if_write_available".
>
> - The '\' was left off of a '\n' in the following call to
> slapi_log_error in slapi_ldap_init_ext():
>
> slapi_log_error(SLAPI_LOG_FATAL, "slapi_ldap_init_ext",
> "failed: unable to set minimum TLS
> protocol level to SSL3n");
>
> - The next call to slapi_log_error is also missing the '\n' in
> slapi_ldap_init_ext().
>
> The patch looks good aside from the above nit-picks (though it's a
> large patch, so I could have missed something).
Thanks. I fixed these, and pushed to master.
>
> -NGK
>> ------------------------------------------------------------------------
>>
>> --
>> 389-devel mailing list
>> 389-devel at redhat.com
>> https://www.redhat.com/mailman/listinfo/fedora-directory-devel
>>
>
> ------------------------------------------------------------------------
>
> --
> 389-devel mailing list
> 389-devel at redhat.com
> https://www.redhat.com/mailman/listinfo/fedora-directory-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3258 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/fedora-directory-devel/attachments/20090707/99bda1d7/attachment.bin>
More information about the Fedora-directory-devel
mailing list