Summary: | [PATCH] bsd.port.mk: Allow builds to fail when package dependencies do not exist | ||
---|---|---|---|
Product: | Ports & Packages | Reporter: | Chris Rees <crees> |
Component: | Individual Port(s) | Assignee: | Bryan Drewery <bdrewery> |
Status: | Closed FIXED | ||
Severity: | Affects Only Me | ||
Priority: | Normal | ||
Version: | Latest | ||
Hardware: | Any | ||
OS: | Any |
Description
Chris Rees
2013-07-21 18:50:02 UTC
Responsible Changed From-To: freebsd-ports-bugs->portmgr bsd.port.mk is portmgr territory (via the GNATS Auto Assign Tool) Responsible Changed From-To: portmgr->bdrewery I'll take it. State Changed From-To: open->closed Committed, with minor changes. Thanks! Author: bdrewery Date: Sat Aug 31 13:56:08 2013 New Revision: 325807 URL: http://svnweb.freebsd.org/changeset/ports/325807 Log: - Add USE_PACKAGE_DEPENDS_ONLY which will try installing dependencies from existing packages and not fallback on building from source. This is useful for package building tools such as poudriere and tinderbox to avoid building from source and confusing the build log, if a dependency failed to build for some reason. NOTE: USE_PACKAGE_DEPENDS has not changed here. It has always reverted to source if the package was not present. PR: ports/180725 Submitted by: crees With hat: portmgr Modified: head/CHANGES head/Mk/bsd.port.mk Modified: head/CHANGES ============================================================================== --- head/CHANGES Sat Aug 31 13:26:26 2013 (r325806) +++ head/CHANGES Sat Aug 31 13:56:08 2013 (r325807) @@ -10,6 +10,13 @@ in the release notes and/or placed into All ports committers are allowed to commit to this file. +20130831: +AUTHOR: bdrewery@FreeBSD.org + + crees has added USE_PACKAGE_DEPENDS_ONLY which works like + USE_PACKAGE_DEPENDS but will not fallback on source if a + package is missing. + 20130731: AUTHOR: bapt@FreeBSD.org Modified: head/Mk/bsd.port.mk ============================================================================== --- head/Mk/bsd.port.mk Sat Aug 31 13:26:26 2013 (r325806) +++ head/Mk/bsd.port.mk Sat Aug 31 13:56:08 2013 (r325807) @@ -1084,8 +1084,11 @@ FreeBSD_MAINTAINER= portmgr@FreeBSD.org # a different checksum and you intend to verify if # the port still works with it. # USE_PACKAGE_DEPENDS -# - Install dependencies from existing packages instead -# of building the port from scratch. +# - Try to install dependencies from existing packages instead +# of building the port from scratch. Fallback on source +# if an existing package is not present. +# USE_PACKAGE_DEPENDS_ONLY +# - Like USE_PACKAGE_DEPENDS, but do not fallback on source. # INSTALL_AS_USER # - Define this to install as the current user, intended # for systems where you have no root access. @@ -4906,7 +4909,7 @@ _DEPEND_ALWAYS= 0 .endif _INSTALL_DEPENDS= \ - if [ X${USE_PACKAGE_DEPENDS} != "X" ]; then \ + if [ -n "${USE_PACKAGE_DEPENDS}" -o -n "${USE_PACKAGE_DEPENDS_ONLY}" ]; then \ subpkgfile=`(cd $$dir; ${MAKE} $$depends_args -V PKGFILE)`; \ subpkgname=$${subpkgfile%-*} ; \ subpkgname=$${subpkgname\#\#*/} ; \ @@ -4920,6 +4923,10 @@ _INSTALL_DEPENDS= \ else \ ${PKG_ADD} $${subpkgfile}; \ fi; \ + elif [ -n "${USE_PACKAGE_DEPENDS_ONLY}" ]; then \ + ${ECHO_MSG} "===> ${PKGNAME} depends on package: $${subpkgfile} - not found"; \ + ${ECHO_MSG} "===> USE_PACKAGE_DEPENDS_ONLY set - will not build from source"; \ + exit 1; \ else \ (cd $$dir; ${MAKE} -DINSTALLS_DEPENDS $$target $$depends_args) ; \ fi; \ _______________________________________________ svn-ports-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-ports-all To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org" Author: bdrewery Date: Tue Sep 3 12:50:47 2013 New Revision: 326143 URL: http://svnweb.freebsd.org/changeset/ports/326143 Log: - Followup to r325807, with USE_PACKAGE_DEPENDS_ONLY, still allow ports framework to use the port for *_DEPENDS= ${NONEXISTENT}:PORT:target dependencies An example port is irc/gseen.mod, it depends on another port being extracted, not an installed pkg. PR: ports/180725 Reported by: antoine With hat: portmgr Modified: head/Mk/bsd.port.mk Modified: head/Mk/bsd.port.mk ============================================================================== --- head/Mk/bsd.port.mk Tue Sep 3 12:50:18 2013 (r326142) +++ head/Mk/bsd.port.mk Tue Sep 3 12:50:47 2013 (r326143) @@ -4923,7 +4923,7 @@ _INSTALL_DEPENDS= \ else \ ${PKG_ADD} $${subpkgfile}; \ fi; \ - elif [ -n "${USE_PACKAGE_DEPENDS_ONLY}" ]; then \ + elif [ -n "${USE_PACKAGE_DEPENDS_ONLY}" -a "$${target}" = "${DEPENDS_TARGET}" ]; then \ ${ECHO_MSG} "===> ${PKGNAME} depends on package: $${subpkgfile} - not found"; \ ${ECHO_MSG} "===> USE_PACKAGE_DEPENDS_ONLY set - will not build from source"; \ exit 1; \ _______________________________________________ svn-ports-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-ports-all To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org" |