[BUG] gropdf, tbl: Completely broken table (was: Ping^1: Chapters of the manual (was: Bug#1018737: ...))

[Alejandro Colomar <alx.manpages@xxxxxxxxx>]

Hi Deri,

On 12/17/22 12:51, Alejandro Colomar wrote:
> > > I noticed a few small issues:
> > Hi Alex,
> >
> > > -  link (.so) pages appear misplaced in the navigation.  For example
> > > writev.2 appears inside man2type.
> > >
> > > -  intro(*) should be the first page of each section.  (So far, subsections
> > > don't have intro, but I wouldn't discard that.)
> >
> > Both fixed in latest version. Which you can find here:-
> >
> > http://chuzzlewit.co.uk/LinuxManBook.pdf

Another bug report (but not about the script; this seems to be about tbl(1) 
interaction with gropdf(1), I guess):

<http://chuzzlewit.co.uk/LinuxManBook.pdf#pdf%3Abm11813>

Running all the linters I know doesn't trigger any warnings on the page source:

$ touch ./man7/suffixes.7
$ make lint
LINT (groff)	tmp/lint/man7/suffixes.7.lint-man.groff.touch
LINT (mandoc)	tmp/lint/man7/suffixes.7.lint-man.mandoc.touch
LINT (tbl)	tmp/lint/man7/suffixes.7.lint-man.tbl.touch
$ touch ./man7/suffixes.7
$ make lint V=1
LINT (groff)	tmp/lint/man7/suffixes.7.lint-man.groff.touch
tbl man7/suffixes.7 \
| eqn -Tutf8  \
| troff -man -t -M ./etc/groff/tmac -m checkstyle -rCHECKSTYLE=3 -ww -Tutf8 
-rLL=78n  \
| grotty -c  \
| col -b -p -x  \
| (! grep -n '.\{80\}.' | sed 's,^,man7/suffixes.7:,' >&2)
touch tmp/lint/man7/suffixes.7.lint-man.groff.touch
LINT (mandoc)	tmp/lint/man7/suffixes.7.lint-man.mandoc.touch
! (mandoc -man -Tlint  man7/suffixes.7 2>&1 \
   | grep -v 'STYLE: lower case character in document title:' \
   | grep -v 'UNSUPP: ignoring macro in table:' \
   | grep -v 'WARNING: cannot parse date, using it verbatim: TH (date)' \
   | grep -v 'WARNING: empty block: UR' \
   ||:; \
) \
| grep '.' >&2
touch tmp/lint/man7/suffixes.7.lint-man.mandoc.touch
LINT (tbl)	tmp/lint/man7/suffixes.7.lint-man.tbl.touch
if grep -q '^\.TS$' man7/suffixes.7 && ! head -n1 man7/suffixes.7 | grep -q '\\" 
t$'; \
then \
	2>&1 echo "man7/suffixes.7:1: missing '\\\" t' comment:"; \
	2>&1 head -n1 <man7/suffixes.7; \
	exit 1; \
fi
if head -n1 man7/suffixes.7 | grep -q '\\" t$' && ! grep -q '^\.TS$' 
man7/suffixes.7; \
then \
	2>&1 echo "man7/suffixes.7:1: spurious '\\\" t' comment:"; \
	2>&1 head -n1 <man7/suffixes.7; \
	exit 1; \
fi
if tail -n+2 <man7/suffixes.7 | grep -q '\\" t$'; \
then \
	2>&1 echo "man7/suffixes.7: spurious '\\\" t' not in first line:"; \
	2>&1 grep -n '\\" t$' man7/suffixes.7 /dev/null; \
	exit 1; \
fi
touch tmp/lint/man7/suffixes.7.lint-man.tbl.touch



Okay, I have a few disabled because they were noisy, but I guess they shouldn't 
be important.  You can see the mandoc(1) ignored warnings just from the above 
command.  Here are the groff ones:


$ cat ./etc/groff/tmac/checkstyle.tmac
.am an-style-warn
.	ds LANDMINE\"
..
.\"
.de end-of-input-macro
.	if d LANDMINE .ab found style problems; aborting
..
.\"
.em end-of-input-macro
.\"
.de an-blank-line-trap
.	sp
..


Cheers,

Alex


--
<http://www.alejandro-colomar.es/>

Attachment: OpenPGP_signature
Description: OpenPGP digital signature