Hi,
Nigel Taylor wrote on Sat, Nov 05, 2011 at 12:09:33PM +0000:
> On 11/05/11 11:18, Remi Pointel wrote:
>> ok rpointel@ if you add USE_GROFF=Yes, if not we have:
>> % man hydra
>> /usr/local/man/man1/hydra.1:106:2: FATAL: line scope broken, syntax violated
> I missed cvs add for the patch to the man page, so it was missed out
> in the diff.
>
> Index: patches/patch-hydra_1
> ===================================================================
> RCS file: patches/patch-hydra_1
> diff -N patches/patch-hydra_1
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-hydra_1 2 Nov 2011 22:35:41 -0000
> @@ -0,0 +1,12 @@
> +$OpenBSD$
> +Correct syntax error
> +--- hydra.1.orig Sun Oct 2 09:06:15 2011
> ++++ hydra.1 Wed Nov 2 22:34:20 2011
> +@@ -102,7 +102,6 @@ verbose mode / show login+pass combination for each at
> + .TP
> + .B \-h, \-\-help
> + Show summary of options.
> +-.TP
> + .SH SEE ALSO
> + .BR xhydra (1),
> + .BR pw-inspector (1).
Sure, that's a syntax error in the manual, but it was also a
long-standing bug in mandoc(1) that this was handled as FATAL.
A few day ago, i have written a patch for it.
I have now committed the mandoc patch to OpenBSD without waiting
for OKs, because people kept running into this particular bug.
So, in the future, such patches are no longer needed in the ports
tree - but they still make sense upstream, of course.
Yours,
Ingo
CVSROOT: /cvs
Module name: src
Changes by: schwa...@cvs.openbsd.org 2011/11/05 10:02:18
Modified files:
usr.bin/mandoc : libman.h man.c man_macro.c man_validate.c
mandoc.h read.c
Log message:
When the HEAD scope of .TP is broken by another block macro,
do not abort with a FATAL error, but report a regular ERROR,
remove the broken .TP from the syntax tree, and prod on.
Reported repeatedly by ports people, at least by brad@ and jeremy@.
Also fixes rendition(4) in Xenocara.
