Package: cdtool
Version: 2.1.8-release-5
Severity: minor
Tags: patch

Dear Maintainer,

   * What led up to the situation?

  Warning about only one argument for the two-fonts macro "IR".

  The patch is in the attachment.

  Summary:

Input file is cdtool.1

  Spelling: give -> given

  Remove space characters at the end of lines.

  Separate the roman part from the only argument of the macro "IR".

  Use macros instead of the escapes "\fX".

  Change '-' to '\-' for options.

  Shorten lines longer than 80 characters.

  Begin sentences on a new line when only one space character is in
front of them.

  Add space in front of a tab to get the formatting correct (troff).

###

 MORE DETAILS

Test nr. 1:

Remove space at end of lines.

Use "git apply ... --whitespace=fix" to fix extra space issues, or use
global configuration "core.whitespace".

16:\fBcdplay\fR    [-[0..9]|-d device]  
[start|stop|resume|eject|close|info|dir] 
24:The 
28:The 
35:In particular, it lists the lengths of all tracks. 
42:If one of these files contains an entry for the current compact disc, 
43:.I cdir 
74:The 
88:then skip the number of tracks given by the second argument forward or 
99:command sets the output volume level of the CD player.  The level is an 
102:The 
121:.B ~/.cdtooldb 
124:.B CDTOOLDBPATH 
125:- colon delimited path of database files to search.  It is not necessary to 
126:specify 
127:.BR ~/.cdtooldb , 
130:.B CDTOOLDEV 

#####

Test nr. 2:

Enable and fix warnings from 'test-groff'.

Input file is /tmp/cdtool.1

<cdtool.1>:41 (macro IR): 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.

41:.IR workman(1).

#####

Test nr. 20:

Use a macro to change to the italic font, instead of \fI [1], if
possible.
The macros have the italic corrections, but "\c" removes them
  or
add the italic corrections.
[1] man-pages(7) [package "manpages"]

14:\fBcdplay\fR    [-[0..9]|-d device]  [\fIstart\fR [\fIstop\fR]
15:\fBcdplay\fR    [-[0..9]|-d device]  [\fI+|-\fR \fInumber\fR]
19:\fBcdvolume\fR  [-[0..9]|-d device]  [-V|-?|-h|\fIlevel\fR]
107:may be changed in \fIconfig.h\fR. Options supported by all programs are:

#####

Test nr. 24:

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

11:\fBcdir\fR -n   [-[0..9]|-d device]  [-r|-s|-t|-V|-?|-h]
46:.IP \fB-n\fR
48:.IP \fB-s\fR
51:.IP \fB-t\fR
62:.IP \fB-a\fR
64:.IP \fB-r\fR
66:.IP \fB-s\fR
68:.IP \fB-t\fR
70:.IP \fB-v\fR
111:.IP "\fB-d device\fR"
115:.IP "\fB-V\fR"
117:.IP "\fB-D\fR"

#####

Test nr. 27:

Split lines longer than 80 characters into two or more lines.
Appropriate break points are the end of a sentence and a subordinate
clause.

cdtool.1: line 53       length 83

#####

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" (.!?).

53:information if available. If blank lines are filled in, this can be appended 
to the
84:starting and stopping track numbers. If the first argument is a
89:backward. If no second argument is give, skip one. A
107:may be changed in \fIconfig.h\fR. Options supported by all programs are:
146:Daniel R. Risacher  <risac...@worldnet.att.net>

#####

Test nr. 30:

Surround a block of comments with the macros ".ig" and "..".
The .\" (\#) at the beginning of each line is then not needed.
Makes it easier to add and remove text and adjust length of lines.

NO PATCH

2:.\" manual page copyright 1994 thomas insel
3:.\" copyright 1995,96 sven oliver moll
4:.\" copyright 1997, 1998 Wade Hampton and Dan Risacher

#####

-- 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.82-1u3 (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 cdtool depends on:
ii  libc6  2.27-3

cdtool recommends no packages.

cdtool suggests no packages.

-- no debconf information

-- 
Bjarni I. Gislason

Attachment: bug.cdtool.1
Description: Unix manual page

Reply via email to