ports/117488: [patch] sysutils/screen enable installation of man page

[Pietro Cerutti]

>Number:         117488
>Category:       ports
>Synopsis:       [patch] sysutils/screen enable installation of man page
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Thu Oct 25 15:10:02 UTC 2007
>Closed-Date:
>Last-Modified:
>Originator:     Pietro Cerutti
>Release:        6.2-STABLE
>Organization:
Bern University of Applied Sciences
>Environment:
FreeBSD gahrtop.localhost 6.2-STABLE FreeBSD 6.2-STABLE #3: Wed Oct 24 08:30:39 CEST 2007     root at gahrtop.localhost:/usr/obj/usr/src/sys/MSI1034  i386

>Description:
sysutils/screen doesn't handle the MAN option correctly: the man page is never installed
>How-To-Repeat:
cd /usr/ports/sysutils/screen
make WITH_MAN=yes install
find /usr/local/man/man1/ -name "screen*"
>Fix:
Apply this patch to the port's Makefile.
Note: a similar problem exists with the INFO option: the INFO doesn't get installed. I'm planning to solve that issue in a future PR, but since INFO pages seem to be much less used than MAN pages, I think this patch deserves higher attention.

--- Makefile.orig       2007-10-25 17:00:50.000000000 +0200 +++ Makefile    2007-10-25 16:58:22.000000000 +0200
@@ -24,11 +24,6 @@
                XTERM_256 "Enable support for 256 colour xterm" OFF

 GNU_CONFIGURE= yes
-.if defined(WITH_MAN)
-MAN1=          screen.1
-.else
-MAKE_ARGS+=    -DNO_MAN
-.endif
 .if defined(WITH_INFO)
 .if !exists(/usr/bin/install-info)
 RUN_DEPENDS+=  install-info:${PORTSDIR}/print/texinfo
@@ -42,6 +37,12 @@

 .include <bsd.port.pre.mk>

+.if defined(WITH_MAN) +MAN1=          screen.1
+.else
+MAKE_ARGS+=    -DNO_MAN
+.endif
+
 .if ${ARCH} == "alpha"
 CFLAGS+=       -O0
 .endif @@ -72,5 +73,8 @@
        @if [ ! -f ${PREFIX}/etc/screenrc ]; then \                 ${CP} ${EXAMPLESDIR}/screenrc ${PREFIX}/etc/screenrc ; \
        fi
+.if defined(WITH_MAN)
+       ${INSTALL_MAN} ${WRKSRC}/doc/screen.1 ${MANPREFIX}/man/man1
+.endif

 .include <bsd.port.post.mk>


>Release-Note:
>Audit-Trail:
>Unformatted:
 >> no output <<