On 07/07/2016 06:45 PM, Michal Židek wrote:
> The man page looks good to me with exception
> for one detail (see inline)
>
> On 07/04/2016 12:45 PM, Pavel Březina wrote:
>> + <refsect1 id='options'>
>> + <title>COMMON OPTIONS</title>
>> + <para>
>> + Those options are available with all commands.
>> + </para>
>> + <variablelist remap='IP'>
>> + <varlistentry>
>> + <term>
>> + <option>--debug</option>
>> + <replaceable>LEVEL</replaceable>
>> + </term>
>> + <xi:include
>>
xmlns:xi="http://www.w3.org/2001/XInclude"
>> href="include/debug_levels.xml" />
>
> This include contains some irrelevant information for sssctl
> that could be confusing for users. But I do not think we
> should block the patch, I will create a ticket to track this
> as a man page bug.
>
>> + </varlistentry>
>> + </variablelist>
>> + </refsect1>
>> +
>> + <xi:include
xmlns:xi="http://www.w3.org/2001/XInclude"
>> href="include/seealso.xml" />
>> +
>
> The attached patch is the same as Pavel's, with some
> whitespaces removed (git complaint about them).
>
> Otherwise LGTM ( == ACK, because we do not have much
> time and I think it is better than nothing).
>
> Michal
The mailman is very slow today for some mails.
Sending again with the hope it will arrive sooner.
Michal