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

Bug#1099832: marked as done (fixnt.1: Some remarks and a patch with editorial changes for this man page)

Your message dated Wed, 27 Aug 2025 07:49:06 +0000
with message-id <E1urAu2-0088zi-0X@fasolo.debian.org>
and subject line Bug#1099832: fixed in a2ps 1:4.15.7-1
has caused the Debian Bug report #1099832,
regarding fixnt.1: Some remarks and a patch with editorial changes for this man page
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
1099832: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1099832
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: a2ps
Version: 1:4.15.6-1+b1
Severity: minor
Tags: patch

   * What led up to the situation?

     Checking for defects with a new version

test-[g|n]roff -mandoc -t -K utf8 -rF0 -rHY=0 -rCHECKSTYLE=10 -ww -z < "man page"

  [Use "groff -e ' $' -e '\\~$' <file>" to find obvious trailing spaces.]

  ["test-groff" is a script in the repository for "groff"; is not shipped]
(local copy and "troff" slightly changed by me).

  [The fate of "test-nroff" was decided in groff bug #55941.]

   * What was the outcome of this action?


troff:<stdin>:6: warning: trailing space in the line
troff:<stdin>:8: warning: trailing space in the line
troff:<stdin>:12: warning: trailing space in the line
troff:<stdin>:17: warning: trailing space in the line
troff:<stdin>:19: warning: trailing space in the line


   * What outcome did you expect instead?

     No output (no warnings).

-.-

  General remarks and further material, if a diff-file exist, are in the
attachments.


-- System Information:
Debian Release: trixie/sid
  APT prefers testing
  APT policy: (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 6.12.12-amd64 (SMP w/2 CPU threads; PREEMPT)
Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), LANGUAGE not set
Shell: /bin/sh linked to /usr/bin/dash
Init: sysvinit (via /sbin/init)

Versions of packages a2ps depends on:
ii  file       1:5.45-3+b1
ii  libc6      2.40-7
ii  libgc1     1:8.2.8-1
ii  libpaper2  2.2.5-0.3+b1
ii  psutils    3.3.8-1

Versions of packages a2ps recommends:
ii  bzip2                     1.0.8-6
pn  lpr | rlpr | cups-client  <none>
ii  wdiff                     1.2.2-7

Versions of packages a2ps suggests:
ii  emacsen-common                       3.0.5
ii  ghostscript                          10.04.0~dfsg-2+b1
ii  groff                                1.23.0-7
ii  gv                                   1:3.7.4-2+b2
pn  html2ps                              <none>
ii  imagemagick                          8:7.1.1.43+dfsg1-1
ii  imagemagick-7.q16 [imagemagick]      8:7.1.1.43+dfsg1-1
pn  t1-cyrillic                          <none>
ii  texlive-binaries [texlive-base-bin]  2024.20240313.70630+ds-5+b1

-- no debconf information

Input file is fixnt.1

Output from "mandoc -T lint  fixnt.1": (shortened list)

      8 whitespace at end of input line

Remove trailing space with: sed -e 's/  *$//'

-.-.

Output from "test-nroff -mandoc -t -ww -z fixnt.1": (shortened list)

      5 trailing space in the line

Remove trailing space with: sed -e 's/  *$//'

-.-.

Remove space characters (whitespace) at the end of lines.
Use "git apply ... --whitespace=fix" to fix extra space issues, or use
global configuration "core.whitespace".

Number of lines affected is

8

-.-.

Put a subordinate sentence (after a comma) on a new line.

fixnt.1:12:files, that are incompatible with psutils.  
fixnt.1:14:is a filter that fixes these problems, allowing the use of
fixnt.1:28:does not check for NTPSOct94.  For a workaround, use a
fixnt.1:31:to replace 'NTPSOct94' with 'NTPSOct95', like so:
fixnt.1:42:Report bugs to the Authors, but avoid sending large postscript files.

-.-.

Remove quotes when there is a printable
but no space character between them
and the quotes are not for emphasis (markup),
for example as an argument to a macro.

fixnt.1:1:.TH "fixnt" 1 "February 2003" "a2ps" "Debian"

-.-.

Space character after a macro call.

43:.P 

-.-.

Trailing space in a macro call.
Remove with "sed -i -e 's/  *$//'"

5:.B fixnt 
7:.I BADFILE.ps  

-.-.

Section headings (.SH and .SS) do not need quoting their arguments.

45:.SH "SEE ALSO"

-.-.

Output from "test-groff  -mandoc -t -K utf8 -rF0 -rHY=0 -rCHECKSTYLE=10 -ww -z ":

troff:<stdin>:6: warning: trailing space in the line
troff:<stdin>:8: warning: trailing space in the line
troff:<stdin>:12: warning: trailing space in the line
troff:<stdin>:17: warning: trailing space in the line
troff:<stdin>:19: warning: trailing space in the line

-.-.

Generally:

Split (sometimes) lines after a punctuation mark; before a conjunction.

--- fixnt.1	2025-03-08 15:57:23.131841621 +0000
+++ fixnt.1.new	2025-03-08 16:11:45.151423256 +0000
@@ -1,22 +1,24 @@
-.TH "fixnt" 1 "February 2003" "a2ps" "Debian"
+.TH fixnt 1 "February 2003" a2ps Debian
 .SH NAME
 fixnt \- Filter for the Windows NT postscript printer driver.
 .SH SYNOPSIS
-.B fixnt 
-< 
-.I BADFILE.ps  
->  
+.B fixnt
+<
+.I BADFILE.ps
+>
 .I GOODFILE.ps
 .SH DESCRIPTION
 The Windows NT postscript driver has a tendency to make broken postscript
-files, that are incompatible with psutils.  
+files,
+that are incompatible with psutils.
 .B fixnt
-is a filter that fixes these problems, allowing the use of
+is a filter that fixes these problems,
+allowing the use of
 .BR psnup (1).
 .PP
-The filter takes the broken postscript file on 
+The filter takes the broken postscript file on
 .BR stdin ,
-and outputs a fixed postscript file on 
+and outputs a fixed postscript file on
 .BR stdout .
 It has no other form for invocation and takes no options on the command-line.
 .SH OPTIONS
@@ -25,10 +27,13 @@ It has no other form for invocation and
 takes no options.
 .SH BUGS
 .B fixnt
-does not check for NTPSOct94.  For a workaround, use a
+does not check for NTPSOct94.
+For a workaround,
+use a
 .BR sed (1)
 command
-to replace 'NTPSOct94' with 'NTPSOct95', like so:
+to replace 'NTPSOct94' with 'NTPSOct95',
+like so:
 .RS
 sed 's/NTPSOct94/NTPSOct95/g'
 .RE
@@ -39,9 +44,11 @@ was written by Holger Bauer <Holger.Baue
 Michael Rath <rath@itsm.uni-stuttgart.de>,
 and Akim Demaille <demaille@inf.enst.fr>.
 .SH REPORTING BUGS
-Report bugs to the Authors, but avoid sending large postscript files.
-.P 
-Patches are always welcome; send to <bauer@itsm.uni-stuttgart.de>.
-.SH "SEE ALSO"
+Report bugs to the Authors,
+but avoid sending large postscript files.
+.P
+Patches are always welcome;
+send to <bauer@itsm.uni-stuttgart.de>.
+.SH SEE ALSO
 .BR psnup (1),
 .BR sed (1)

  Any program (person), that produces man pages, should check the output
for defects by using (both groff and nroff)

[gn]roff -mandoc -t -ww -b -z -K utf8 <man page>

  The same goes for man pages that are used as an input.

  For a style guide use

  mandoc -T lint

-.-

  Any "autogenerator" should check its products with the above mentioned
'groff', 'mandoc', and additionally with 'nroff ...'.

  It should also check its input files for too long (> 80) lines.

  This is just a simple quality control measure.

  The "autogenerator" may have to be corrected to get a better man page,
the source file may, and any additional file may.

  Common defects:

  Not removing trailing spaces (in in- and output).
  The reason for these trailing spaces should be found and eliminated.

  "git" has a "tool" to point out whitespace,
see for example "git-apply(1)" and git-config(1)")

  Not beginning each input sentence on a new line.
Line length and patch size should thus be reduced.

  The script "reportbug" uses 'quoted-printable' encoding when a line is
longer than 1024 characters in an 'ascii' file.

  See man-pages(7), item "semantic newline".

-.-

The difference between the formatted output of the original and patched file
can be seen with:

  nroff -mandoc <file1> > <out1>
  nroff -mandoc <file2> > <out2>
  diff -d -u <out1> <out2>

and for groff, using

\"printf '%s\n%s\n' '.kern 0' '.ss 12 0' | groff -mandoc -Z - \"

instead of 'nroff -mandoc'

  Add the option '-t', if the file contains a table.

  Read the output from 'diff -d -u ...' with 'less -R' or similar.

-.-.

  If 'man' (man-db) is used to check the manual for warnings,
the following must be set:

  The option \"-warnings=w\"

  The environmental variable:

export MAN_KEEP_STDERR=yes (or any non-empty value)

  or

  (produce only warnings):

export MANROFFOPT=\"-ww -b -z\"

export MAN_KEEP_STDERR=yes (or any non-empty value)

-.-

--- End Message ---
--- Begin Message --- 
Source: a2ps
Source-Version: 1:4.15.7-1
Done: Andreas Tille <tille@debian.org>

We believe that the bug you reported is fixed in the latest version of
a2ps, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 1099832@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Andreas Tille <tille@debian.org> (supplier of updated a2ps package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Format: 1.8
Date: Wed, 27 Aug 2025 09:25:52 +0200
Source: a2ps
Architecture: source
Version: 1:4.15.7-1
Distribution: unstable
Urgency: medium
Maintainer: Debian QA Group <packages@qa.debian.org>
Changed-By: Andreas Tille <tille@debian.org>
Closes: 971818 971822 971930 1096269 1099525 1099832
Changes:
 a2ps (1:4.15.7-1) unstable; urgency=medium
 .
   * QA upload.
 .
   [ Andreas Tille ]
   * New upstream version
     Closes: #1096269
   * Move package to Salsa
   * d/copyright: DEP5
   * Standards-Version: 4.7.2 (routine-update)
   * Replace transitional package pkg-config by pkgconf in Build-Depends
     (routine-update)
   * Trim trailing whitespace.
   * Add a missing ',' between the option and the following explanation
     Closes: #971818
   * Fix some formatting in the manual
     Closes: #971822
   * Editorial changes for composeglyphs.1
     Closes: #1099525
   * Editorial changes for fixnt.1
     Closes: #1099832
   * a2ps-lpr-wrapper.1: fix some typographic issues
     Closes: #971930
Checksums-Sha1:
 ffa33d9b08f58165bca55e97a683f2367a767508 2290 a2ps_4.15.7-1.dsc
 86ab078578b0ed7e07f9a9c1c276b5079e1bc49e 3615522 a2ps_4.15.7.orig.tar.gz
 68c6d4502ee014dc9a8d6d9107a60d8ae578ddd4 488 a2ps_4.15.7.orig.tar.gz.asc
 57ac58116babd2091f51d17bebe4b55d1c71734c 21404 a2ps_4.15.7-1.debian.tar.xz
 23eadf979be1f7ee9c093ac9ca32ac2d103940a0 11847 a2ps_4.15.7-1_amd64.buildinfo
Checksums-Sha256:
 24708beac67e0b019010e215f7640a3cbf7400c0dd817d564e50dfd3f769ff47 2290 a2ps_4.15.7-1.dsc
 715f38670afd950b4ca71c01f468feefad265ca52d3f112934c63c0a8bfbb8af 3615522 a2ps_4.15.7.orig.tar.gz
 991bb565e23584d5bc519d81868039a5429521ef18b224e0af95fd51805aa481 488 a2ps_4.15.7.orig.tar.gz.asc
 a83109388d3680ca4ec0f262e94f1d3cd1a918f793f5043d41e52e3b273ebb61 21404 a2ps_4.15.7-1.debian.tar.xz
 7fb995013beae444e7908590b51c0a15e39cd7426aec92ea80dda084b294c870 11847 a2ps_4.15.7-1_amd64.buildinfo
Files:
 9889473ff907fa473b13d73f71ce9443 2290 text optional a2ps_4.15.7-1.dsc
 c0b2187a56f9d60931227dc4678cbaea 3615522 text optional a2ps_4.15.7.orig.tar.gz
 186d7cc2b4501a21605625db789b7c3b 488 text optional a2ps_4.15.7.orig.tar.gz.asc
 07dfd4a1d1d18485a6aa5684565ae448 21404 text optional a2ps_4.15.7-1.debian.tar.xz
 782c3c048aee0b084e2b8124f7d912f2 11847 text optional a2ps_4.15.7-1_amd64.buildinfo

-----BEGIN PGP SIGNATURE-----

iQJFBAEBCgAvFiEE8fAHMgoDVUHwpmPKV4oElNHGRtEFAmius+wRHHRpbGxlQGRl
Ymlhbi5vcmcACgkQV4oElNHGRtFhOA//SQor2iIRKpmuyy9H63N/dWRGwql6wIYd
7aav30EhSPGo8fxRSFpRxMyKZLwDAZCyZj8pG1HizQ4o2vvqrUOWIJvPkAm3Yeat
1v+1vTVE+XlrOPr3+cMOtAWoTgmpyllvKpBfuJ4Rj64JJA1BJrky2rd5FCuYizam
GzeMdhpK6YvTjOrUdG++mY+FvskGwYwnWhU5bm+E4zdXtM+rDyFGPc4O+d3av23M
YcBx++sLgWkg4J4ED6Q+LpwTgrlwH0Y+8WDBrwdY1UH0gZ1A7LrVQFmWNolqAm07
Be+3f39+1qYa996MdYOhYBzNzNtfoKwdDS1nYCqr/3hjkS2YIDFlGVDhVB/FRUYs
qPsfQnsrQeh2bOD1H/cDBlf3H4lddWcOsONnFOdURe63PrKycqQw68wsheWKyWnF
Ch0gDnAh/95AbF+6zeWW+qvm88EznFWSxlmYvs+bb0Ri4JTDuvKfcIgEe3OVziJH
l0rVHL0s2GhJT3xh7EgwaZZyN5MpIyN9agn7ZxoD1H6aqChX97KRaiybvA+3GY58
mOzVfWIhVgQj2H2votN/i2MvVzfkJdUpn8uPwEMbDwVhXpOvGzQFQA/9J2AmUUnV
NEDoSbrp+P3l4VQBV/mypWbmK38YTc+YMZgBg8OJincSxAg/AqVcrYqBRwHVqyUQ
6zhMFRg1W9k=
=X7Bg
-----END PGP SIGNATURE-----

Attachment: pgpPOT_2QSNKm.pgp
Description: PGP signature


--- End Message ---
Reply to: