Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

man: fix some single-quoted strings #2828

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

a3a3el
Copy link
Contributor

@a3a3el a3a3el commented Mar 10, 2025

Commit 721aadc ("man: change quoting style") replaced single-quoted strings of the form:

'lorem ipsum'

with:

`lorem ipsum'

to prevent the possibility of single quotes appearing at the beginning of lines, because these will be misinterpreted by troff. However, changes to the man-page sources in 1.12.9 reintroduced some of these, including one which does appear at the beginning of a line in mu-query(7):

'"', '*', '(' and ')'. The details are shell-specific. In case of doubt, the

which results in the following warning:

troff:<standard input>:25: warning: macro '"',' not defined

and the omission of that line from the rendered paragraph:

NOTE:  if  you  use  queries on the command-line (say, for mu find), you
need to quote any characters that would otherwise be interpreted by  the
shell, such as *--analyze option can be useful.

Use back-ticks for the opening quotes.

Fixes: 5e2b7d5 ("mu-query.7: update documentation")

a3a3el added 2 commits March 10, 2025 21:23
Commit 721aadc ("man: change quoting style") replaced single-quoted strings
of the form:

    'lorem ipsum'

with:

    `lorem ipsum'

to prevent the possibility of single quotes appearing at the beginning of lines,
because these will be misinterpreted by troff.  However, changes to the man-page
sources in 1.12.9 reintroduced some of these, including one which does appear at
the beginning of a line in mu-query(7):

    '"', '*', '(' and ')'. The details are shell-specific. In case of doubt, the

which results in the following warning:

    troff:<standard input>:25: warning: macro '"',' not defined

and the omission of that line from the rendered paragraph:

    NOTE:  if  you  use  queries on the command-line (say, for mu find), you
    need to quote any characters that would otherwise be interpreted by  the
    shell, such as *--analyze option can be useful.

Use back-ticks for the opening quotes.

Fixes: 5e2b7d5 ("mu-query.7: update documentation")
Signed-off-by: Jeremy Sowden <[email protected]>
Otherwise:

    (with the \040 specifying a space in the regular expression, and and extra '\' to escape it)

is rendered as:

    (with the $ 40 specifying a space in the regular expression, and and extra `$´ to escape it)

Fixes: 5e2b7d5 ("mu-query.7: update documentation")
Signed-off-by: Jeremy Sowden <[email protected]>
@a3a3el
Copy link
Contributor Author

a3a3el commented Mar 10, 2025

Just noticed another problem, which is fixed by the second commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant