From owner-freebsd-ports-bugs@FreeBSD.ORG Wed Aug 28 17:10:01 2013 Return-Path: Delivered-To: freebsd-ports-bugs@smarthost.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 6651AF97 for ; Wed, 28 Aug 2013 17:10:01 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:1900:2254:206c::16:87]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 479582884 for ; Wed, 28 Aug 2013 17:10:01 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.7/8.14.7) with ESMTP id r7SHA14E066892 for ; Wed, 28 Aug 2013 17:10:01 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.7/8.14.7/Submit) id r7SHA168066891; Wed, 28 Aug 2013 17:10:01 GMT (envelope-from gnats) Resent-Date: Wed, 28 Aug 2013 17:10:01 GMT Resent-Message-Id: <201308281710.r7SHA168066891@freefall.freebsd.org> Resent-From: FreeBSD-gnats-submit@FreeBSD.org (GNATS Filer) Resent-To: freebsd-ports-bugs@FreeBSD.org Resent-Reply-To: FreeBSD-gnats-submit@FreeBSD.org, Chris Rees Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id 983C3EB9 for ; Wed, 28 Aug 2013 17:01:58 +0000 (UTC) (envelope-from crees@bayofrum.net) Received: from mk-outboundfilter-6.mail.uk.tiscali.com (mk-outboundfilter-6.mail.uk.tiscali.com [212.74.114.14]) by mx1.freebsd.org (Postfix) with ESMTP id C27A8282C for ; Wed, 28 Aug 2013 17:01:57 +0000 (UTC) Received: from host-2-102-106-185.as13285.net (HELO pegasus.bayofrum.net) ([2.102.106.185]) by smtp.pipex.tiscali.co.uk with ESMTP; 28 Aug 2013 18:01:45 +0100 Received: by pegasus.bayofrum.net (Postfix, from userid 1001) id C2D4B326FE; Wed, 28 Aug 2013 18:01:23 +0100 (BST) Message-Id: <20130828170124.C2D4B326FE@pegasus.bayofrum.net> Date: Wed, 28 Aug 2013 18:01:23 +0100 (BST) From: Chris Rees To: FreeBSD-gnats-submit@freebsd.org X-Send-Pr-Version: 3.113 Subject: ports/181624: Fix port: textproc/docbook-420 dependencies are incorrectly expressed X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: Chris Rees List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 28 Aug 2013 17:10:01 -0000 >Number: 181624 >Category: ports >Synopsis: Fix port: textproc/docbook-420 dependencies are incorrectly expressed >Confidential: no >Severity: serious >Priority: medium >Responsible: freebsd-ports-bugs >State: open >Quarter: >Keywords: >Date-Required: >Class: sw-bug >Submitter-Id: current-users >Arrival-Date: Wed Aug 28 17:10:00 UTC 2013 >Closed-Date: >Last-Modified: >Originator: Chris Rees >Release: FreeBSD 9.1-RELEASE-p4 amd64 >Organization: >Environment: System: FreeBSD pegasus.bayofrum.net 9.1-RELEASE-p4 FreeBSD 9.1-RELEASE-p4 #1 r252227M: Sat Jun 29 13:55:30 BST 2013 root@pegasus.bayofrum.net:/usr/obj/usr/src/sys/PEGASUS amd64 >Description: http://www.bayofrum.net/medusa/tinderlogs/9.1-local-20130828122206/docbook-4.2.log When installing docbook-420 it depends on an unzip executable, which is found a /usr/bin/unzip, but it actually uses UNZIP_CMD, which is defined in bsd.commands.mk as ${LOCALBASE}/bin/unzip. Also, we do not mute installation commands. While here, remove useless check for directory's existence (mkdir -p does not complain if it exists). >How-To-Repeat: >Fix: --- docbook-420-dependencies.diff begins here --- Index: Makefile =================================================================== --- Makefile (revision 325543) +++ Makefile (working copy) @@ -1,12 +1,5 @@ -# New ports collection makefile for: docbook-420 -# Date created: 29 Oct 2003 -# Whom: kutulu -# -# Based on murray's docbook-410 port -# which is based on niks' docbook-310 port -# +# Created by: kutulu # $FreeBSD$ -# PORTNAME= docbook PORTVERSION= 4.2 @@ -20,7 +13,8 @@ RUN_DEPENDS= ${ISOCAT}:${PORTSDIR}/textproc/iso8879 \ ${XMLCATMGR}:${PORTSDIR}/textproc/xmlcatmgr -BUILD_DEPENDS= unzip:${PORTSDIR}/archivers/unzip +BUILD_DEPENDS= ${UNZIP_CMD}:${PORTSDIR}/archivers/unzip +INSTALL_DEPENDS=${UNZIP_CMD}:${PORTSDIR}/archivers/unzip USE_ZIP= yes NO_MTREE= yes NO_BUILD= yes @@ -38,11 +32,11 @@ LATEST_LINK= docbook-420 pre-su-install: - @[ -d ${INSTDIR}/${PORTVERSION} ] || ${MKDIR} ${INSTDIR}/${PORTVERSION} + ${MKDIR} ${INSTDIR}/${PORTVERSION} do-install: - @${UNZIP_CMD} ${EXTRACT_BEFORE_ARGS} ${DISTDIR}/${DISTFILES} -d ${INSTDIR}/${PORTVERSION} - @${INSTALL_DATA} ${FILESDIR}/catalog ${INSTDIR}/${PORTVERSION}/catalog + ${UNZIP_CMD} ${EXTRACT_BEFORE_ARGS} ${DISTDIR}/${DISTFILES} -d ${INSTDIR}/${PORTVERSION} + ${INSTALL_DATA} ${FILESDIR}/catalog ${INSTDIR}/${PORTVERSION}/catalog @${ECHO_CMD} "CATALOG \"${ISOCAT}\"" >> ${INSTDIR}/${PORTVERSION}/catalog @${CHMOD} ${SHAREMODE} ${INSTDIR}/${PORTVERSION}/* @${CHOWN} -R ${SHAREOWN}:${SHAREGRP} ${INSTDIR} --- docbook-420-dependencies.diff ends here --- -- This message has been scanned for viruses and dangerous content by MailScanner, and is believed to be clean. >Release-Note: >Audit-Trail: >Unformatted: