Package: unrar
Version: 1:5.5.8-1
Severity: minor
Tags: patch

Dear Maintainer,

   * What led up to the situation?

Some warnings from "test-groff" (development version of "groff"):

<unrar.1>:9 (macro BR): only 1 argument, but more are expected
<unrar.1>:21 (macro BR): only 1 argument, but more are expected
<unrar.1>:42 (macro BR): only 1 argument, but more are expected

  The patch is in the attachment.

  Summary:

  See the manual "man-pages.7" in the package "man-pages" (Debian)
about the use of bold and italic words.

Change ".BR <one argument>" to ".B ..."

Protect a full stop (.) with '\&' if it begins or ends a string and does
not mean an end of a sentence.

Change '-' to '\-' if in front of an option.

Move a sentence to the next line, if only one space is in front of it.

###

DETAILS:

Input file is unrar.1

Test nr. 2:

Enable and fix warnings from 'test-groff'.

<unrar.1>:9 (macro BR): only 1 argument, but more are expected
<unrar.1>:21 (macro BR): only 1 argument, but more are expected
<unrar.1>:42 (macro BR): only 1 argument, but more are expected

chk_manuals: Output is from: test-groff -b -e -mandoc -T utf8 -rF0 -t -w w -z 

  and

Test nr. 15:

Change the name of a macro for two fonts (e.g., BR and IR) to one
letter, if there is only one argument.
Add the second argument if needed.  It is sometimes part of the first
one.

9:.BR unrar
21:.BR unrar
42:.BR NOTE:

#####

Test nr. 8:

Protect a full stop (.) with "\&", if it has a blank (white-space) in
front of or (ignoring characters transparent to it) after it, and it
does not mean an end of a sentence.

6:.I "<command> [-<switch 1> -<switch N>] archive [files...] [path\...]"

#####

Test nr. 24:

Change a HYPHEN-MINUS (code 0x55, 2D) to a minus (\-), if in front of a
name for an option.

46:.B -av-
49:.B -c-
52:.B -f
55:.B -kb
58:.B -ierr
61:.B -inul
64:.B -o+
67:.B -o-
70:.B -p<password>
73:.B -p-
76:.B -r
79:.B -u
82:.B -v
85:.B -x<file>
88:.B -x@<list>
91:.B -x@
94:.B -y

#####

Test nr. 28:

Wrong distance between sentences or protect the indicator.

1) Separate the sentences and subordinate clauses; each begins on a new
line.  See man-pages(7) [package "manpages"] and "info groff".

Or

2) Adjust space between sentences (two spaces),

3) or protect the indicator by adding "\&" after it.

The "indicator" is an "end-of-sentence character" (.!?).

43:Every switch must be separated by a whitespace. You cannot put them

#####

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

Kernel: Linux 4.9.88-1 (SMP w/2 CPU cores)
Locale: LANG=is_IS.iso88591, LC_CTYPE=is_IS.iso88591 (charmap=ISO-8859-1), 
LANGUAGE=is_IS.iso88591 (charmap=ISO-8859-1)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)

Versions of packages unrar depends on:
ii  libc6       2.27-3
ii  libgcc1     1:8-20180425-1
ii  libstdc++6  8-20180425-1

unrar recommends no packages.

unrar suggests no packages.

-- no debconf information

-- 
Bjarni I. Gislason
--- unrar.1     2017-08-21 12:29:11.000000000 +0000
+++ unrar.1.new 2018-05-05 01:38:06.000000000 +0000
@@ -3,22 +3,23 @@
 unrar \- extract files from rar archives
 .SH SYNOPSIS
 .B unrar
-.I "<command> [-<switch 1> -<switch N>] archive [files...] [path\...]"
+.IR command " [" \-switch " \&...\&] " archive \
+" [" file " \&...\&] [" path " \&...\&]"
 .SH "DESCRIPTION"
 This manual page documents briefly the
-.BR unrar
+.B unrar
 command.
 .br
 This manual page was written for the Debian GNU/Linux distribution
 because the original program does not have a manual page.
 .br
 Commands and options described here are as of unrar 2.02.
-.SH OPTIONS
+.SH COMMANDS
 After the program name comes a command and then optional switches with
 dashes before them.
 A summary of commands is included below.
 For a complete description, run
-.BR unrar
+.B unrar
 without options.
 .TP
 .B e
@@ -39,59 +40,59 @@ Verbosely list archive.
 .B x
 Extract files with full path.
 .SH SWITCHES
-.BR NOTE:
-Every switch must be separated by a whitespace. You cannot put them
-together.
+.B NOTE:
+Every switch must be separated by a whitespace.
+You cannot put them together.
 .TP
-.B -av-
+.B \-av\-
 Disable Authenticity Verification check.
 .TP
-.B -c-
+.B \-c\-
 Disable comments show.
 .TP
-.B -f
+.B \-f
 Freshen files.
 .TP
-.B -kb
+.B \-kb
 Keep broken extracted files.
 .TP
-.B -ierr
+.B \-ierr
 Send all messages to stderr.
 .TP
-.B -inul
+.B \-inul
 Disable all messages.
 .TP
-.B -o+
+.B \-o+
 Overwrite existing files.
 .TP
-.B -o-
+.B \-o\-
 Do not overwrite existing files.
 .TP
-.B -p<password>
+.BI \-p password
 Set password.
 .TP
-.B -p-
+.B \-p\-
 Do not query password.
 .TP
-.B -r
+.B \-r
 Recurse subdirectories.
 .TP
-.B -u
+.B \-u
 Update files.
 .TP
-.B -v
+.B \-v
 List all volumes.
 .TP
-.B -x<file>
+.BI \-x file
 Exclude specified file.
 .TP
-.B -x@<list>
+.BI \-x@ list
 Exclude files in specified list file.
 .TP
-.B -x@
+.B \-x@
 Read file names to exclude from stdin.
 .TP
-.B -y
+.B \-y
 Assume Yes on all queries.
 .SH AUTHOR
 This manual page was written by Petr Cech <c...@debian.org> according

Reply via email to