diff mbox

Problems in dash.1, sh.1, sh.distrib.1

Message ID 20180619152737.7BBCA3A4F09@snark.thyrsus.com (mailing list archive)
State Accepted
Delegated to: Herbert Xu
Headers show

Commit Message

esr@thyrsus.com June 19, 2018, 3:27 p.m. UTC
This is automatically generated email about markup problems in a man
page for which you appear to be responsible.  If you are not the right
person or list, please tell me so I can correct my database.

See http://catb.org/~esr/doclifter/bugs.html for details on how and
why these patches were generated.  Feel free to email me with any
questions.  Note: These patches do not change the modification date of
any manual page.  You may wish to do that by hand.

I apologize if this message seems spammy or impersonal. The volume of
markup bugs I am tracking is over five hundred - there is no real
alternative to generating bugmail from a database and template.

--
                             Eric S. Raymond
Problems with dash.1:

(Identical patches should apply to: sh.1 sh.distrib.1)

Ambiguous or invalid backslash.  This doesn't cause groff a problem.
but it confuses doclifter and may confuse older troff implementations.

Comments

Herbert Xu Aug. 29, 2018, 3:37 a.m. UTC | #1
On Tue, Jun 19, 2018 at 11:27:37AM -0400, esr@thyrsus.com wrote:
> This is automatically generated email about markup problems in a man
> page for which you appear to be responsible.  If you are not the right
> person or list, please tell me so I can correct my database.
> 
> See http://catb.org/~esr/doclifter/bugs.html for details on how and
> why these patches were generated.  Feel free to email me with any
> questions.  Note: These patches do not change the modification date of
> any manual page.  You may wish to do that by hand.
> 
> I apologize if this message seems spammy or impersonal. The volume of
> markup bugs I am tracking is over five hundred - there is no real
> alternative to generating bugmail from a database and template.
> ---
>                              Eric S. Raymond
> Problems with dash.1:
> 
> (Identical patches should apply to: sh.1 sh.distrib.1)
> 
> Ambiguous or invalid backslash.  This doesn't cause groff a problem.
> but it confuses doclifter and may confuse older troff implementations.
> 
> --- dash.1-unpatched	2018-05-16 16:52:28.478288208 -0400
> +++ dash.1	2018-05-16 16:52:28.182290220 -0400
> @@ -1825,7 +1825,7 @@
>  .Ic shift
>  will issue an error message, and exit with return status 2.
>  .It test Ar expression
> -.It \&[ Ar expression Cm ]
> +.It \&[ Ar expression Cm \&]

Patch applied but as you can see above I've restored the Cm which
was deleted by your original patch.

Thanks,
diff mbox

Patch

--- dash.1-unpatched	2018-05-16 16:52:28.478288208 -0400
+++ dash.1	2018-05-16 16:52:28.182290220 -0400
@@ -1825,7 +1825,7 @@ 
 .Ic shift
 will issue an error message, and exit with return status 2.
 .It test Ar expression
-.It \&[ Ar expression Cm ]
+.It \&[ Ar expression \&]
 The
 .Ic test
 utility evaluates the expression and, if it evaluates