ports/130664: [PATCH] x11/roxterm: install manpages

[Ben Morrow]

>Number:         130664
>Category:       ports
>Synopsis:       [PATCH] x11/roxterm: install manpages
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Sat Jan 17 23:30:01 UTC 2009
>Closed-Date:
>Last-Modified:
>Originator:     Ben Morrow <ben at morrow.me.uk>
>Release:        FreeBSD 7.1-RELEASE i386
>Organization:
>Environment:
System: FreeBSD osiris.mauzo.dyndns.org 7.1-RELEASE FreeBSD 7.1-RELEASE #1: Thu Jan 8 20:32:44 GMT 2009 root at osiris.mauzo.dyndns.org:/usr/obj/usr/src/sys/OSIRIS i386
>Description:
The ROXTerm port does not currently install the manpages, but it would
be useful to do so. There is also a bug in the configure script that
needs patching before they can be generated.

I'm not sure whether it's correct to only install manpages when
!NOPORTDOCS: is this option for all documentation, or just for extra
docs in HTML and such?
>How-To-Repeat:
Install x11/roxterm. No manpages are installed.
>Fix:

--- roxterm-man.patch begins here ---
diff -Nur roxterm-orig/Makefile roxterm/Makefile
--- roxterm-orig/Makefile	2009-01-17 22:27:26.000000000 +0000
+++ roxterm/Makefile	2009-01-17 22:20:12.000000000 +0000
@@ -20,6 +20,14 @@
 GNU_CONFIGURE=	yes
 
 .if !defined(NOPORTDOCS)
+XSL_DIR=	${LOCALBASE}/share/xsl/docbook
+
+BUILD_DEPENDS=	${XSL_DIR}:${PORTSDIR}/textproc/docbook-xsl \
+		xsltproc:$(PORTSDIR}/textproc/libxslt
+
+CONFIGURE_ARGS+=    --with-docbook-man-xsl=${XSL_DIR}/manpages/docbook.xsl
+MAN1=		roxterm.1 roxterm-config.1
+
 INSTALL_TARGET+=install install-doc-local
 .endif
 
diff -Nur roxterm-orig/files/patch-configure roxterm/files/patch-configure
--- roxterm-orig/files/patch-configure	1970-01-01 01:00:00.000000000 +0100
+++ roxterm/files/patch-configure	2009-01-17 22:20:12.000000000 +0000
@@ -0,0 +1,11 @@
+--- configure	2009-01-17 21:39:24.000000000 +0000
++++ configure	2009-01-17 21:39:41.000000000 +0000
+@@ -20208,7 +20208,7 @@
+ 
+ # Check whether --with-docbook-man-xsl was given.
+ if test "${with_docbook_man_xsl+set}" = set; then
+-  withval=$with_docbook_man_xsl; if test "x$withval" != "x" -a test -f "$withval"
++  withval=$with_docbook_man_xsl; if test "x$withval" != "x" -a -f "$withval"
+ 	then
+ 		DOCBOOK_XSL="$withval"
+ 	else
--- roxterm-man.patch ends here ---


>Release-Note:
>Audit-Trail:
>Unformatted: