Subject: bin/36271: /usr/share/groff_font/devhtml/DESC is missing a required parameter
To: None <gnats-admin@netbsd.org, netbsd-bugs@netbsd.org>
From: Greg A. Woods <woods@planix.com>
List: netbsd-bugs
Date: 05/03/2007 22:30:01
>Number:         36271
>Category:       bin
>Synopsis:       /usr/share/groff_font/devhtml/DESC is missing a required parameter
>Confidential:   no
>Severity:       serious
>Priority:       high
>Responsible:    bin-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Thu May 03 22:30:01 +0000 2007
>Originator:     Greg A. Woods
>Release:        netbsd-current 2007/05/02
>Organization:
Planix, Inc.; Toronto, Ontario; Canada
>Environment:
System: NetBSD (anything since 2.0)
>Description:

	try to run "pre-grohtml", or "groff -Thtml" and find that it
	just complains loudly:

		pre-grohtml:fatal error: devhtml/DESC must set the image_generator field, exiting

	Groff itself doesn't actually fail with a fatal exit code, but
	with the pre-grohtml preprocessor dead, all input is swallowed
	and no output is generated.

>How-To-Repeat:

	pre-grohtml
	groff -Thtml

>Fix:

	this is a bit of a hack, but it seems to work OK

	the choice of "gs" is based on the default suggested for
	unix-like platforms -- it'll work as expected once one of the
	Ghostscript modules from pkgsrc is installed, but without it
	doesn't cause any immediate harm

	there's also a rule entry that doesn't start with a tab....

Index: gnu/usr.bin/groff/font/Makefile.tty
===================================================================
RCS file: /cvs/master/m-NetBSD/main/src/gnu/usr.bin/groff/font/Makefile.tty,v
retrieving revision 1.7
diff -u -r1.7 Makefile.tty
--- gnu/usr.bin/groff/font/Makefile.tty	21 Oct 2003 10:01:20 -0000	1.7
+++ gnu/usr.bin/groff/font/Makefile.tty	3 May 2007 22:06:23 -0000
@@ -15,7 +15,7 @@
 $(FONTS): R.proto
 	${_MKTARGET_CREATE}
 	(charwidth=`expr $(RES) / $(CPI)` ; \
- 	 sed -e "s/^name [A-Z]*$$/name ${.TARGET}/" \
+	 sed -e "s/^name [A-Z]*$$/name ${.TARGET}/" \
 	     -e "s/^\\([^	]*\\)	[0-9]+	/\\1	$$charwidth	/" \
 	     -e "s/^spacewidth [0-9]+$$/spacewidth $$charwidth/" \
 	     -e "s/^internalname .*$$/internalname $@/" \
@@ -32,3 +32,6 @@
 	    -e "s/^vert .*$$/vert `expr $(RES) / $(LPI)`/" \
 	    -e "s/^fonts .*$$/fonts `set $(FONTS); echo $$#` $(FONTS)/" \
 	    ${.ALLSRC} >${.TARGET}
+.if ${DIST_SUBDIR} == "font/devhtml"
+	@echo "image_generator gs" >> ${.TARGET}
+.endif