Re: [PATCH 4/4] docs: setterm.1 clean up manual page groff style

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 25 May 2014 22:10, Benno Schulenberg <bensberg@xxxxxxxxxxxxx> wrote:
> On Sun, May 25, 2014, at 12:58, Sami Kerola wrote:
>> That convention should be documented. Meanwhile I hope the following
>> is a bit closer how the manual should be.
>>
>> New version:
>> https://github.com/kerolasa/lelux-utiliteetit/commit/bbc05cc2e30a6d6a3adcc7207d5549b782f03574
>
>> -.B force
>> +.I force
>
> This and poke are literal arguments and should stay bold.

Thank you, I missed them.

https://github.com/kerolasa/lelux-utiliteetit/commit/bbc05cc2e30a6d6a3adcc7207d5549b782f03574

>> -.B \-\-default
>> +\fB\-\-default\fP
>
> Why do you make this kind of changes?  The first is perfectly fine
> and more readable.

I find it more readable when options after .TP are always using
\fB...\fR, what is the only way to make optional option with arguments
and other complex cases to work. Making the simple cases look a like
the other cases feels correct to me.

> I am not going to make any more detailed comments, it's much
> easier to later simply provide a patch.

Would you mind writing the howto update in same go?

-- 
Sami Kerola
http://www.iki.fi/kerolasa/
--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux