From owner-svn-ports-head@FreeBSD.ORG Mon Jul 28 18:11:57 2014 Return-Path: Delivered-To: svn-ports-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 818DCE1D; Mon, 28 Jul 2014 18:11:57 +0000 (UTC) Received: from duck.symmetricom.us (duck.symmetricom.us [206.168.13.214]) by mx1.freebsd.org (Postfix) with ESMTP id 58C882EB1; Mon, 28 Jul 2014 18:11:57 +0000 (UTC) Received: from gromit.timing.com (gromit.timing.com [206.168.13.209]) by duck.symmetricom.us (8.14.6/8.14.6) with ESMTP id s6SIBu8s078117; Mon, 28 Jul 2014 12:11:56 -0600 (MDT) (envelope-from jhein@microsemi.com) Received: from gromit.timing.com (localhost [127.0.0.1]) by gromit.timing.com (8.14.8/8.14.5) with ESMTP id s6SIBmmF047217; Mon, 28 Jul 2014 12:11:48 -0600 (MDT) (envelope-from jhein@gromit.timing.com) Received: (from jhein@localhost) by gromit.timing.com (8.14.8/8.14.8/Submit) id s6SIBmPG047216; Mon, 28 Jul 2014 12:11:48 -0600 (MDT) (envelope-from jhein) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Message-ID: <21462.37476.571859.625026@gromit.timing.com> Date: Mon, 28 Jul 2014 12:11:48 -0600 From: John Hein To: Antoine Brodin Subject: @cwd ${LOCALBASE} for ldconfig files breaks prefix - Re: svn commit: r357076 - head/Mk In-Reply-To: <201406082125.s58LPt1q050276@svn.freebsd.org> References: <201406082125.s58LPt1q050276@svn.freebsd.org> X-Mailer: VM 8.2.0b-trunk-1500 under 24.3.1 (i386-portbld-freebsd8.3) Cc: svn-ports-head@freebsd.org, svn-ports-all@freebsd.org, ports-committers@freebsd.org X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 28 Jul 2014 18:11:57 -0000 Antoine Brodin wrote at 21:25 +0000 on Jun 8, 2014: > Author: antoine > Date: Sun Jun 8 21:25:54 2014 > New Revision: 357076 > URL: http://svnweb.freebsd.org/changeset/ports/357076 > QAT: https://qat.redports.org/buildarchive/r357076/ > > Log: > Kill NO_LDCONFIG_MTREE, it is long dead > Make USE_LDCONFIG work when PREFIX!=LOCALBASE, > LDCONFIG_DIR and LDCONFIG_32DIR are expected in LOCALBASE > > Phabric: D195 > Reviewed by: bapt > With hat: portmgr > > Modified: > head/Mk/bsd.port.mk > > Modified: head/Mk/bsd.port.mk > ============================================================================== > --- head/Mk/bsd.port.mk Sun Jun 8 21:22:25 2014 (r357075) > +++ head/Mk/bsd.port.mk Sun Jun 8 21:25:54 2014 (r357076) > @@ -970,12 +970,6 @@ FreeBSD_MAINTAINER= portmgr@FreeBSD.org > # - Same as USE_LDCONFIG but the target file is > # ${PREFIX}/libdata/ldconfig32/${UNIQUENAME} instead. > # Note: that should only be used on 64-bit architectures. > -# NO_LDCONFIG_MTREE > -# - Denotes whether the libdata/ldconfig directory is part of > -# the mtree on a given OSVERSION system. If it is not, we > -# create the directory, pull in the ldconfig_compat port, > -# and clean up on de-installation. NOTE: this variable is > -# internal to bsd.port.mk and must not be set in your Makefile. > # > # DOCSDIR - Name of the directory to install the packages docs in. > # Default: ${PREFIX}/share/doc/${PORTNAME} > @@ -4022,18 +4016,15 @@ install-ldconfig-file: > -${LDCONFIG} -m ${USE_LDCONFIG} > .endif > .endif > -.if ${USE_LDCONFIG} != "${PREFIX}/lib" && !defined(INSTALL_AS_USER) > +.if ${USE_LDCONFIG} != "${LOCALBASE}/lib" && !defined(INSTALL_AS_USER) > @${ECHO_MSG} "===> Installing ldconfig configuration file" > -.if defined(NO_LDCONFIG_MTREE) > - @${MKDIR} ${STAGEDIR}${PREFIX}/${LDCONFIG_DIR} > +.if defined(NO_MTREE) > + @${MKDIR} ${STAGEDIR}${LOCALBASE}/${LDCONFIG_DIR} > .endif > @${ECHO_CMD} ${USE_LDCONFIG} | ${TR} ' ' '\n' \ > - > ${STAGEDIR}${PREFIX}/${LDCONFIG_DIR}/${UNIQUENAME} > - @${ECHO_CMD} "@cwd ${PREFIX}" >> ${TMPPLIST} > + > ${STAGEDIR}${LOCALBASE}/${LDCONFIG_DIR}/${UNIQUENAME} > + @${ECHO_CMD} "@cwd ${LOCALBASE}" >> ${TMPPLIST} > @${ECHO_CMD} ${LDCONFIG_DIR}/${UNIQUENAME} >> ${TMPPLIST} > -.if defined(NO_LDCONFIG_MTREE) > - @${ECHO_CMD} "@unexec rmdir ${LDCONFIG_DIR} >/dev/null 2>&1 || true" >> ${TMPPLIST} > -.endif > .endif > .endif > .endif > @@ -4049,16 +4040,13 @@ install-ldconfig-file: > .endif > .if !defined(INSTALL_AS_USER) > @${ECHO_MSG} "===> Installing 32-bit ldconfig configuration file" > -.if defined(NO_LDCONFIG_MTREE) > - @${MKDIR} ${STAGEDIR}${PREFIX}/${LDCONFIG_32DIR} > +.if defined(NO_MTREE) > + @${MKDIR} ${STAGEDIR}${LOCALBASE}/${LDCONFIG_32DIR} > .endif > @${ECHO_CMD} ${USE_LDCONFIG32} | ${TR} ' ' '\n' \ > - > ${STAGEDIR}${PREFIX}/${LDCONFIG32_DIR}/${UNIQUENAME} > - @${ECHO_CMD} "@cwd ${PREFIX}" >> ${TMPPLIST} > + > ${STAGEDIR}${LOCALBASE}/${LDCONFIG32_DIR}/${UNIQUENAME} > + @${ECHO_CMD} "@cwd ${LOCALBASE}" >> ${TMPPLIST} > @${ECHO_CMD} ${LDCONFIG32_DIR}/${UNIQUENAME} >> ${TMPPLIST} > -.if defined(NO_LDCONFIG_MTREE) > - @${ECHO_CMD} "@unexec rmdir ${LDCONFIG32_DIR} >/dev/null 2>&1" >> ${TMPPLIST} > -.endif > .endif > .endif > .if defined(INSTALLS_SHLIB) This (@cwd ${LOCALBASE}) breaks PREFIX for things in the plist after ldconfig entries (e.g., info entries). There should perhaps be an @cwd to return to previous prefix after the addition of the ldconfig entries? For example: https://redports.org//~jhein/20140728060045-61045-227980/tex-kpathsea-6.1.0_2.log