Package: sensible-utils
Version: 0.0.24
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 ' $' <file>" to find 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>:15: 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.6-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)
-- no debconf information
Input file is sensible-browser.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
-.-
So any 'generator' should check its products with the above mentioned
'groff', 'mandoc', and additionally with 'nroff ...'.
This is just a simple quality control measure.
The 'generator' may have to be corrected to get a better man page,
the source file may, and any additional file may.
Common defects:
Input text line longer than 80 bytes.
Not removing trailing spaces (in in- and output).
The reason for these trailing spaces should be found and eliminated.
Not beginning each input sentence on a new line.
Lines should thus be shorter.
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 -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 of 'diff -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)
-.-.
Output from "mandoc -T lint sensible-browser.1": (shortened list)
1 input text line longer than 80 bytes
1 whitespace at end of input line
-.-.
Output from "test-groff -mandoc -t -ww -z sensible-browser.1": (shortened list)
1 trailing space in the line
-.-.
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
1
-.-.
Split lines longer than 80 characters into two or more lines.
Appropriate break points are the end of a sentence and a subordinate
clause; after punctuation marks.
Line 24, length 84
Documentation of behavior of sensible-utils under a debian system is available
under
-.-.
Put a parenthetical sentence, phrase on a separate line,
if not part of a code.
See man-pages(7), item "semantic newline".
Not considered in a patch, too many lines.
sensible-browser.1:26:/usr/share/doc/debian-policy (you might need to install
debian-policy)
-.-.
Put a subordinate sentence (after a comma) on a new line.
14:environment variable could be set, and will be used if set.
-.-.
Remove quotes when there is no space character between them
and the quotes are not for emphasis (markup),
for example as an argument to a macro.
2:.TH SENSIBLE-BROWSER 1 "12 Jan 2020" "Debian"
23:.SH "STANDARD"
-.-.
Output from "test-groff -mandoc -t -K utf8 -rF0 -rHY=0 -rCHECKSTYLE=10 -ww -z
":
troff:<stdin>:15: warning: trailing space in the line
--- sensible-browser.1 2025-01-08 00:20:13.940658284 +0000
+++ sensible-browser.1.new 2025-01-08 00:27:06.148466423 +0000
@@ -1,5 +1,5 @@
.\" -*- nroff -*-
-.TH SENSIBLE-BROWSER 1 "12 Jan 2020" "Debian"
+.TH SENSIBLE-BROWSER 1 "12 Jan 2020" Debian
.SH NAME
sensible-browser \- sensible web browsing
.SH SYNOPSIS
@@ -11,16 +11,20 @@ Programs in Debian can use this script
as their default web browser or emulate their behavior.
.br
.B BROWSER
-environment variable could be set, and will be used if set.
-Any string acceptable as a command_string operand to the
+environment variable could be set,
+and will be used if set.
+Any string acceptable as a command_string operand to the
.B sh \-c
-command shall be valid (this is the same behavior of the
+command shall be valid
+(this is the same behavior of the
.B EDITOR
variable as documented in
.BR \%environ (7)).
.SH "SEE ALSO"
.BR environ (7)
-.SH "STANDARD"
-Documentation of behavior of sensible-utils under a debian system is available
under
-section 11.4 of debian-policy usually installed under
-/usr/share/doc/debian-policy (you might need to install debian-policy)
+.SH STANDARD
+Documentation of behavior of sensible-utils under a debian system is
+available under section 11.4 of debian-policy
+usually installed under /usr/share/doc/debian-policy
+(you might need to install debian-policy)
+
