All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marko Myllynen <myllynen-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
To: "Michael Kerrisk (man-pages)"
	<mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Mike Frysinger <vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org>
Cc: linux-man <linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: charmap.5: clarify keyword syntax
Date: Tue, 21 Jun 2016 10:07:42 +0300	[thread overview]
Message-ID: <5768E7BE.6020002@redhat.com> (raw)
In-Reply-To: <60df9e3e-8ea2-0c70-c00e-af07f36b0019-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

Hi,

On 2016-06-20 23:34, Michael Kerrisk (man-pages) wrote:
> On 06/15/2016 09:58 AM, Marko Myllynen wrote:
>> On 2016-06-14 19:56, Mike Frysinger wrote:
>>> On 13 Jun 2016 10:20, Marko Myllynen wrote:
>>>> The patch below updates charmap.5 to match the syntax all the glibc
>>>> charmap files are using currently.
>>>
>>> hmm, i guess this is confusing.  when i read the man page, i see it as
>>> the normal syntax of "<required value>" rather than the <> being literal
>>> characers that you need to type out.  although in that regard, i would
>>> expect it to look something like:
>>> 	<character> <byte-sequence> [comment]
>>>
>>> although in the locale(5) page, we rarely use the <foo> syntax.
>>
>> Yes, in the locale(5) page <foo> is now used only in cases like:
>>
>>   The characters <space> and <tab> are automatically included.
>>
>> In the glibc charmap files the <> notation is used with keywords, the
>> man pages states that comment is optional but would it make thing
>> clearer if we would use something like:
>>
>> .TP
>> <
>> .I code_set_name
>>> is followed by the name of the character map.
>>
>> (Ok, that renders badly but you get the idea.)
> 
> I'd take patches to fix this. 
> 
> I think you want:
> 
> .TP
> .RI < code_set_name >

Thanks, the patch below seems to do the trick:

---
 man5/charmap.5 | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

diff --git a/man5/charmap.5 b/man5/charmap.5
index f43d5a5..9bc2440 100644
--- a/man5/charmap.5
+++ b/man5/charmap.5
@@ -28,30 +28,30 @@ can use charmaps to create locale variants for different character sets.
 The charmap file starts with a header that may consist of the
 following keywords:
 .TP
-.I <code_set_name>
+.RI < code_set_name >
 is followed by the name of the character map.
 .TP
-.I <comment_char>
+.RI < comment_char >
 is followed by a character that will be used as the comment character
 for the rest of the file.
 It defaults to the number sign (#).
 .TP
-.I <escape_char>
+.RI < escape_char >
 is followed by a character that should be used as the escape character
 for the rest of the file to mark characters that should be interpreted
 in a special way.
 It defaults to the backslash (\\).
 .TP
-.I <mb_cur_max>
+.RI < mb_cur_max >
 is followed by the maximum number of bytes for a character.
 The default value is 1.
 .TP
-.I <mb_cur_min>
+.RI < mb_cur_min >
 is followed by the minimum number of bytes for a character.
 This value must be less than or equal than
-.IR <mb_cur_max> .
+.RI < mb_cur_max >.
 If not specified, it defaults to
-.IR <mb_cur_max> .
+.RI < mb_cur_max >.
 .PP
 The character set definition section starts with the keyword
 .I CHARMAP
@@ -60,12 +60,12 @@ in the first column.
 The following lines may have one of the two following forms to
 define the character set:
 .TP
-.I <character> byte-sequence comment
+.RI < character >\  byte-sequence\ comment
 This form defines exactly one character and its byte sequence,
 .I comment
 being optional.
 .TP
-.I <character>..<character> byte-sequence comment
+.RI < character >..< character >\  byte-sequence\ comment
 This form defines a character range and its byte sequence,
 .I comment
 being optional.
@@ -89,10 +89,10 @@ in the first column.
 The following lines may have one of the two following forms to
 define the widths of the characters:
 .TP
-.I <character> width
+.RI < character >\  width
 This form defines the width of exactly one character.
 .TP
-.I <character>...<character> width
+.RI < character >...< character >\  width
 This form defines the width for all the characters in the range.
 .PP
 The width definition section ends with the string

Cheers,

-- 
Marko Myllynen
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2016-06-21  7:07 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-06-13  7:20 charmap.5: clarify keyword syntax Marko Myllynen
     [not found] ` <575E5EA5.6050707-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-06-14 16:31   ` Michael Kerrisk (man-pages)
2016-06-14 16:56   ` Mike Frysinger
     [not found]     ` <20160614165643.GF4053-UgUKS2FnFs9+urZeOPWqwQ@public.gmane.org>
2016-06-15  7:58       ` Marko Myllynen
     [not found]         ` <57610A8D.6080105-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-06-20 20:34           ` Michael Kerrisk (man-pages)
     [not found]             ` <60df9e3e-8ea2-0c70-c00e-af07f36b0019-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2016-06-21  7:07               ` Marko Myllynen [this message]
     [not found]                 ` <5768E7BE.6020002-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
2016-06-23 10:55                   ` Michael Kerrisk (man-pages)

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5768E7BE.6020002@redhat.com \
    --to=myllynen-h+wxahxf7alqt0dzr+alfa@public.gmane.org \
    --cc=linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=vapier-aBrp7R+bbdUdnm+yROfE0A@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.