Re: [PATCH 4/4] man/*.{1,1m,3x,5,7}: Soft-code ncurses version in `TH` call.

[Thomas Dickey]

On Sat, Mar 16, 2024 at 04:35:34PM -0500, G. Branden Robinson wrote:
> Hi Thomas,
> 
> The attachments did the trick.
> 
> At 2024-03-16T17:17:03-0400, Thomas Dickey wrote:
> > see attached (I'm referring to the checks/warnings, not the normal 
> > operation)
> 
> > #!/bin/sh
> > nroff -b -man -w all -z tic.1m
> 
> Thanks.  It was the `-w all`.
> 
> I can reproduce it now.
> 
> > troff: backtrace: '/usr/share/groff/1.23.0/tmac/an.tmac':512: while loop
> > troff: backtrace: '/usr/share/groff/1.23.0/tmac/an.tmac':516: macro 
> > 'an*scan-string-for-backslash'
> > troff: backtrace: '/usr/share/groff/1.23.0/tmac/an.tmac':543: macro 
> > 'an*abbreviate-inner-footer'
> > troff: backtrace: '/usr/share/groff/1.23.0/tmac/an.tmac':340: macro 'TH'
> > troff: backtrace: file 'tic.1m':31
> > troff:tic.1m:31: warning: missing closing delimiter in output comparison 
> > operator (got a newline)
> 
> Yeah, that's my footer abbreviator all right (a new groff 1.23 feature,
> to say nothing of the suggestive path, so thanks to RMS for once for
> mandating the versioned subdirectory that so many people hate).
> 
> I don't get any diagnostics with the _installed_ page, which is all I
> normally check per my previous email.  (Attaching my script.)
> 
> $ ~/groff-stable/bin/nroff -ww -man ~/ncurses-HEAD/share/man/man1/tic.1 2>&1 
> >/dev/null | grep . || echo no output
> no output
> 
> $ sed -n '31p' ~/ncurses-HEAD/share/man/man1/tic.1
> .TH tic 1 2023-12-30 "ncurses 6.4" "User commands"
> 
> You've probably found a bug in one of the two internal macros named
> above, and since I wrote those macros, it'll certainly be my fault.
> 
> You can quiet it back down with the `-Wdelim` option.
> 
> $ ~/groff-stable/bin/nroff -ww -Wdelim -man ./man/tic.1m 2>&1 >/dev/null | 
> grep . || echo no output
> no output
> 
> I'll see about fixing this for groff 1.24.  String processing in *roff
> is a masochist's paradise.
> 
> What would you like to do in the meantime?

well ... I did complete reviewing these changes, but that ate up time
that I'd wanted to spend on the check-size feature.  Since last week,
I've also made a configure-script fix (half-tested), and see gcc 14
surface (with its own problems), and have made a workaround for a
serious regression (caused by others...) on one platform, which ought
to be checked in the configure script.

Deferring this into tomorrow impacts other things that I wanted to do
tomorrow (fixing a bug in xterm, and working around a minefield which
some other developers have thoughtfully provided).

So considering it all, I'm inclined to push this off to next weekend,
using the remainder of today to work on the check-size feature.

-- 
Thomas E. Dickey <dickey@invisible-island.net>
https://invisible-island.net

signature.asc
Description: PGP signature