On Tue, May 12, 2015 at 03:42:47PM +0200, Pavel Reichl wrote:
>>>> From 3adee0c2ff207eef7c2edcfed80eb47754570210 Mon
Sep 17 00:00:00 2001
>>>>From: Pavel Reichl <preichl(a)redhat.com>
>>>>Date: Wed, 25 Mar 2015 05:03:12 -0400
>>>>Subject: [PATCH] LDAP: warn about lockout option being deprecated
>>>>
>>>>---
>>>> src/man/sssd-ldap.5.xml | 7 +++++++
>>>> src/providers/ldap/sdap_access.c | 9 ++++++++-
>>>> 2 files changed, 15 insertions(+), 1 deletion(-)
>>>>
>>>>diff --git a/src/man/sssd-ldap.5.xml b/src/man/sssd-ldap.5.xml
>>>>index
00da3964af7619f19525e76c4f1292586dd60f54..b46e07fec96ffa7305ab41ce42c16b2e94e53292 100644
>>>>--- a/src/man/sssd-ldap.5.xml
>>>>+++ b/src/man/sssd-ldap.5.xml
>>>>@@ -1955,6 +1955,13 @@ ldap_access_filter = (employeeType=admin)
>>>> be set for this feature to work.
>>>> </para>
>>>> <para>
>>>>+ <emphasis>
>>>>+ Please note that this option is superseded
by
>>>>+ ppolicy option and will be dropped in next
major
>>>>+ release.
>>>Since we can't remove the option soon, I'd say "might be removed
in a
>>>future release". It would also be nice to point the admin to the option
>>>he should be using now instead.
>>OK, I mentioned ppolicy option for the second time, or did you just want me
>>to rephrase the sentence?
>Ah, sorry, no. I was reading the rendered text and didn't realize
>ppolicy was an option and not a generic term. Sorry, can you remove the
>added sentence adn then just enclose ppolicy in <quote> ? That would
>make the rendering better readable I guess. Also, "the ppolicy option".
OK, please see updated patch.
ACK