On Fri, Feb 12, 2016 at 02:37:05PM +0100, Pavel Reichl wrote:
On 02/12/2016 02:19 PM, Pavel Reichl wrote:
>
>
>On 02/09/2016 04:25 PM, Pavel Reichl wrote:
>>
>>
>>On 02/09/2016 08:09 AM, Jakub Hrozek wrote:
>>>On Mon, Feb 08, 2016 at 01:56:07PM +0100, Pavel Reichl wrote:
>>
>>>>diff --git a/src/man/sssd.conf.5.xml b/src/man/sssd.conf.5.xml
>>>>index
73a21bfa0049bc4d3cfacb49201707868c87e533..2dbc58a451686beda0faa9e9366bbc3b3b4c253e 100644
>>>>--- a/src/man/sssd.conf.5.xml
>>>>+++ b/src/man/sssd.conf.5.xml
>>>>@@ -1040,6 +1040,27 @@ pam_account_expired_message = Account expired,
please call help desk.
>>>> </listitem>
>>>> </varlistentry>
>>>> <varlistentry>
>>>>+ <term>pam_account_locked_message
(string)</term>
>>>>+ <listitem>
>>>>+ <para>
>>>>+ If user is authenticating and
>>>
>>>Please ask someone for an English review (I know Dan started, but I
>>>didn't see a fixed version yet). At the very least, this should read
"a
>>>user".
>>
>>I attached Dan's patch. I took the liberty of adding note regarding pam
verbosity. Hope it's fine by Dan.
>>
>>>
>>>>+ account is locked then by default
>>>>+ 'Permission denied' is output. This
output will
>>>>+ be changed to content of this variable if it
is
>>>>+ set.
>>>>+ </para>
>>>>+ <para>
>>>>+ example:
>>>>+ <programlisting>
>>>>+pam_account_locked_message = Account locked, please call help desk.
>>>>+ </programlisting>
>>>>+ </para>
>>>>+ <para>
>>>>+ Default: none
>>>>+ </para>
>>>>+ </listitem>
>>>>+ </varlistentry>
>>>>+ <varlistentry>
>>>> <term>p11_child_timeout
(integer)</term>
>>>> <listitem>
>>>> <para>
>>>
>>>The rest of the patch looks good to me and seems to work as advertized.
>>
>>Thanks.
>>
>>
>>_______________________________________________
>>sssd-devel mailing list
>>sssd-devel(a)lists.fedorahosted.org
>>https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org
>>
>
>Patch set with amended commit message. Commit message was acked by AB.
>
Lukas asked me for adding comment regarding documenting '775' string to the code.
Please see updated patch set. Thanks.
I think we've reviewed these patches enough :)
ACK
CI link:
http://sssd-ci.duckdns.org/logs/job/37/40/summary.html