Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 13 Mar 2007 10:33:40 +0100 (CET)
From:      Martin Matuska <martin@matuska.org>
To:        FreeBSD-gnats-submit@FreeBSD.org
Subject:   ports/110257: [MAINTAINER] lang/tcl83
Message-ID:  <20070313093340.3CACB3F47A@mail.vx.sk>
Resent-Message-ID: <200703130940.l2D9e62v070560@freefall.freebsd.org>

next in thread | raw e-mail | index | archive | help

>Number:         110257
>Category:       ports
>Synopsis:       [MAINTAINER] lang/tcl83
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          maintainer-update
>Submitter-Id:   current-users
>Arrival-Date:   Tue Mar 13 09:40:05 GMT 2007
>Closed-Date:
>Last-Modified:
>Originator:     Martin Matuska
>Release:        FreeBSD 6.2-RELEASE
>Organization:
>Environment:
FreeBSD 6.2-RELEASE i386 and amd64
>Description:
This update changes the following:
- improved handling of WITH_TCL85_MAN
- manpages are not installed by default (default on for Tcl 8.4)

Portlint -A and tinderbox do not complain.
>How-To-Repeat:
>Fix:
diff -Nbur lang/tcl85.orig/Makefile lang/tcl85/Makefile
--- lang/tcl85.orig/Makefile	Thu Mar  8 15:28:16 2007
+++ lang/tcl85/Makefile	Mon Mar 12 18:34:06 2007
@@ -7,7 +7,7 @@
 
 PORTNAME=	tcl
 PORTVERSION=	8.5.a.5
-PORTREVISION=	2
+PORTREVISION=	3
 CATEGORIES=	lang
 MASTER_SITES=	${MASTER_SITE_SOURCEFORGE}
 MASTER_SITE_SUBDIR=	tcl
@@ -30,7 +30,7 @@
 PLIST_SUB=	TCL_VER=${TCL_VER} SHORT_TCL_VER=85
 EXTRACT_AFTER_ARGS=|${TAR} -xpf - --exclude compat/
 
-OPTIONS=	TCL85_MAN "Install tcl 8.5 manpages" on \
+OPTIONS=	TCL85_MAN "Install tcl 8.5 manpages" off \
 		TCLSH_WRAPPER "Require tclsh wrapper from ports" off
 
 .include <bsd.port.pre.mk>
@@ -53,7 +53,7 @@
 RUN_DEPENDS+=		tclsh:${PORTSDIR}/lang/tcl-tk-wrapper
 .endif
 
-.if defined(NO_INSTALL_MANPAGES) || defined(WITHOUT_TCL85_MAN)
+.if defined(NO_INSTALL_MANPAGES) || !defined(WITH_TCL85_MAN)
 
 PLIST_SUB+=	INSTMAN='@comment '
 
@@ -702,14 +702,6 @@
 	${REINPLACE_CMD} \
 		-e 's,-DNO_MEMMOVE=1,,' -e 's,-DNO_STRING_H=1,,' \
 		-e 's,^COMPAT_OBJS.*,,' ${WRKSRC}/Makefile
-.if defined(NO_INSTALL_MANPAGES) || defined(WITHOUT_TCL85_MAN)
-	${REINPLACE_CMD} -e 's|^MAN_INSTALL_DIR.*$$|MAN_INSTALL_DIR = ${WRKDIR}|' \
-		${WRKSRC}/Makefile
-. for mp in ${MAN1} ${MAN3} ${MANN}
-	@${REINPLACE_CMD} -e 's|^.so man.macros|.so ${DATADIR}/man.macros|'	\
-		${WRKSRC}/../doc/${mp}
-. endfor
-.endif
 .if ${BLACKHOLE} != '' && ${BLACKHOLE} != 0
 	# ====================================================
 	# net.inet.tcp.blackhole is non-zero. Some http-tests
@@ -720,12 +712,14 @@
 .endif
 
 post-install:
-.if exists(${PKGINSTALL})
-	${SETENV} PKG_PREFIX=${PREFIX} ${SH} ${PKGINSTALL}
-.endif
-.if !defined(NO_INSTALL_MANPAGES) && !defined(WITHOUT_TCL85_MAN)
+.if !defined(NO_INSTALL_MANPAGES) && defined(WITH_TCL85_MAN)
 	${MKDIR} ${DATADIR}
 	${INSTALL_DATA} ${WRKSRC}/../doc/man.macros ${DATADIR}
+. for mp in ${MAN1} ${MAN3} ${MANN}
+	@${REINPLACE_CMD} -e 's|^.so man.macros|.so ${DATADIR}/man.macros|'	\
+		${WRKSRC}/../doc/${mp}
+. endfor
+	@cd ${WRKSRC} && ${MAKE} -f Makefile install-doc ${MAKE_ENV}
 .endif
 
 regression-test:	build
diff -Nbur lang/tcl85.orig/files/patch-unix-Makefile.in lang/tcl85/files/patch-unix-Makefile.in
--- lang/tcl85.orig/files/patch-unix-Makefile.in	Mon Jan 29 04:06:56 2007
+++ lang/tcl85/files/patch-unix-Makefile.in	Mon Mar 12 18:28:54 2007
@@ -1,5 +1,5 @@
---- Makefile.in.orig	Wed Jan 17 13:10:55 2007
-+++ Makefile.in	Wed Jan 17 12:55:33 2007
+--- Makefile.in.orig	Mon Oct 23 15:41:32 2006
++++ Makefile.in	Mon Mar 12 18:28:44 2007
 @@ -57,6 +57,8 @@
  
  # Directory in which to install the include file tcl.h:
@@ -44,6 +44,15 @@
  
  ${STUB_LIB_FILE}: ${STUB_LIB_OBJS}
  	rm -f $@
+@@ -706,7 +718,7 @@
+ dltest.marker:
+ 	cd dltest ; $(MAKE)
+ 
+-INSTALL_TARGETS = install-binaries install-libraries install-doc @EXTRA_INSTALL@
++INSTALL_TARGETS = install-binaries install-libraries @EXTRA_INSTALL@
+ 
+ install: $(INSTALL_TARGETS)
+ 
 @@ -735,6 +747,13 @@
  	@echo "Installing $(LIB_FILE) to $(LIB_INSTALL_DIR)/"
  	@@INSTALL_LIB@
>Release-Note:
>Audit-Trail:
>Unformatted:



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20070313093340.3CACB3F47A>