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

Bug#971930: a2ps-lpr-wrapper.1: fix some typographic issues



Control: tags 971930 - patch

On 2020-10-09, Bjarni Ingi Gislason wrote:
> Fix warnings from "mandoc -T lint" (superfluous macro ".PP").
>
> Fix warnings from test-groff (space at the end of an argument).
>
> Reduce space between words.
>
> Begin a sentence on a new line.
>
> Remove an unnecessary second font change.
>
> Use ".BR" for references for a manual page.
>
> ###
>
> Patch:
>
> --- a2ps-lpr-wrapper.1	2020-10-08 20:32:56.000000000 +0000
> +++ a2ps-lpr-wrapper.1.new	2020-10-08 23:24:03.000000000 +0000
> @@ -2,31 +2,27 @@
>  .SH "NAME"
>  a2ps-lpr-wrapper \(em lp/lpr wrapper script for GNU a2ps on Debian
>  .SH "SYNOPSIS"
> -.PP
> -\fBa2ps-lpr-wrapper\fR [\fB-d \fIprinter\fR\fP]  [\fB\fIfiles\fR\fP]
> +\fBa2ps-lpr-wrapper\fR [\fB-d \fIprinter\fR] [\fB\fIfiles\fR]
>  .SH "DESCRIPTION"
> -.PP
>  This manual page documents briefly the
> -\fBa2ps-lpr-wrapper\fR       command.
> +\fBa2ps-lpr-wrapper\fR command.
>  .PP
>  \fBa2ps-lpr-wrapper\fR uses either
>  \fBlp\fP or \fBlpr\fP to send
> -\fIfiles\fR to \fIprinter\fR. It
> -determines which of the two programs is currently installed and adds the
> -appropriate parameters.
> +\fIfiles\fR to \fIprinter\fR.
> +It determines which of the two programs is currently installed and
> +adds the appropriate parameters.
>  .SH "OPTIONS"
> -.IP "\fB-d \fIprinter\fR\fP         " 10
> +.IP "\fB-d \fIprinter\fR" 10
>  Destination system printer.
> -.IP "\fB\fIfiles\fR\fP         " 10
> +.IP \fIfiles\fR 10
>  List of files to be printed.
>  .SH "SEE ALSO"
> -.PP
> -a2ps (1).
> +.BR a2ps (1).
>  .PP
>  The programs are documented fully by the a2ps documentation available via the
>  \fBInfo\fP system.
>  .SH "AUTHOR"
> -.PP
>  This manual page was written by Michael Tautschnig tautschn@model.in.tum.de for
>  the \fBDebian\fP system (but may be used by others).  Permission is
>  granted to copy, distribute and/or modify this document under

Thanks for the patch!

Unfortunately it cannot be applied to the package, as a2ps-lpr-wrapper.1
is generated from debian/manpages/a2ps-lpr-wrapper.sgml:

  https://browse.dgit.debian.org/a2ps.git/tree/debian/manpages/a2ps-lpr-wrapper.sgml

In debian/rules:

  https://browse.dgit.debian.org/a2ps.git/tree/debian/rules#n27


Can you update your patch to apply to the .sgml source?


live well,
  vagrant

Attachment: signature.asc
Description: PGP signature


Reply to: