Date: Thu, 25 Oct 2007 15:06:54 GMT From: Pietro Cerutti <gahr@gahr.ch> To: freebsd-gnats-submit@FreeBSD.org Subject: ports/117488: [patch] sysutils/screen enable installation of man page Message-ID: <200710251506.l9PF6smx029860@www.freebsd.org> Resent-Message-ID: <200710251510.l9PFA2sk042575@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
>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@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 <<
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200710251506.l9PF6smx029860>