devel/srecord sets USE_GROFF=Yes. The build uses groff(1) in three ways:

1. Runs soelim(1) on the manpages, then installs them to man1/3/5.
2. Runs groff -Tascii on the manpages, then installs them to cat1/3/5.
3. Runs groff -Tps on additional documentation, then installs to share/doc.

None of these really need USE_GROFF. In particular, the manpages
installed in step 1 format fine with mandoc(1); I diffed the output
and found nothing problematic. So we can replace USE_GROFF with a
build dependency on groff.

This means the items in step 2 are useless. But we don't have to remove
them, since update-plist apparently ignores them anyway.

ok?

--- devel/srecord/Makefile
+++ devel/srecord/Makefile
@@ -3,7 +3,7 @@ BROKEN-arm=	SIGBUS in ps2pdf
 COMMENT=		collection of tools for manipulating EPROM load files
 
 DISTNAME=		srecord-1.64
-REVISION =		4
+REVISION =		5
 CATEGORIES=		devel
 
 SHARED_LIBS +=  srecord              0.0      # 0.0
@@ -20,11 +20,10 @@ WANTLIB += c gcrypt gpg-error iconv intl m ${COMPILER_
 COMPILER =		base-clang ports-gcc base-gcc
 
 BUILD_DEPENDS=		devel/boost \
-			print/ghostscript/gnu
+			print/ghostscript/gnu \
+			textproc/groff
 LIB_DEPENDS=		security/libgcrypt
 
-USE_GROFF=		Yes
-
 CONFIGURE_STYLE=	gnu
 CONFIGURE_ENV=		CPPFLAGS="-I${LOCALBASE}/include" \
 			LDFLAGS="-L${LOCALBASE}/lib"
blob - 0bdbd343667948b4e8a7a889d374485559647938
blob + 494e5249ac7fb96aae86ba8ec4269d9fd8e4699f
--- devel/srecord/pkg/PLIST
+++ devel/srecord/pkg/PLIST
@@ -153,7 +153,7 @@ include/srecord/quit/prefix.h
 include/srecord/record.h
 include/srecord/srecord.h
 include/srecord/string.h
-lib/libsrecord.a
+@static-lib lib/libsrecord.a
 lib/libsrecord.la
 @lib lib/libsrecord.so.${LIBsrecord_VERSION}
 lib/pkgconfig/srecord.pc