Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 4 Dec 2018 22:36:34 +0000 (UTC)
From:      Matthias Andree <mandree@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r486634 - in head: graphics/ilmbase sysutils/e2fsprogs
Message-ID:  <201812042236.wB4MaYS7009279@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mandree
Date: Tue Dec  4 22:36:34 2018
New Revision: 486634
URL: https://svnweb.freebsd.org/changeset/ports/486634

Log:
  Align *_DEPENDS and use paths for bash in my ports; fix var name.
  
  On systems where bash is in $PATH but not in ${LOCALBASE}/bin/bash,
  ilmbase would fail to build, and e2fsprogs might fail if the
  BASHTESTS option was enabled.
  Make sure that the _DEPENDS and the actual path handed to the configure or
  test rig match to the letter, to avoid a time-of-check vs. time-of-use
  discrepancy.
  
  In e2fsprogs, the helper variable was also mistyped and would never
  amend to BUILD_DEPENDS. Fix typo.
  
  Check my other ports, but wiggle and xmlto [textproc/] seem unaffected.
  
  Reported by:	J R Matthews (direct email)
  MFH:		2018Q4

Modified:
  head/graphics/ilmbase/Makefile
  head/sysutils/e2fsprogs/Makefile

Modified: head/graphics/ilmbase/Makefile
==============================================================================
--- head/graphics/ilmbase/Makefile	Tue Dec  4 20:10:05 2018	(r486633)
+++ head/graphics/ilmbase/Makefile	Tue Dec  4 22:36:34 2018	(r486634)
@@ -11,14 +11,17 @@ COMMENT=	ILM Base libraries a.k.a. Half, IlmThread, Im
 
 LICENSE=	BSD3CLAUSE
 
-BUILD_DEPENDS=	bash:shells/bash
+# there are some systems that have a static /bin/bash, so
+# in order to make BUILD_DEPENDS effective, use the same absolute
+# path we will pass down through CONFIGURE_SHELL below.
+BUILD_DEPENDS=	${BASH_CMD}:shells/bash
 
 USES=		compiler:c++14-lang pkgconfig libtool
 USE_LDCONFIG=	yes
 
 GNU_CONFIGURE=	yes
 # The configure script uses bash arrays
-CONFIGURE_SHELL=	${LOCALBASE}/bin/bash
+CONFIGURE_SHELL=${BASH_CMD}
 CONFIGURE_ARGS=	--disable-static
 TEST_TARGET=	check
 
@@ -26,6 +29,8 @@ PORTDOCS=	AUTHORS ChangeLog README.md
 OPTIONS_DEFINE=		DOCS LARGE_STACK
 LARGE_STACK_DESC=	Enable sys-dependent large stack optimizations
 LARGE_STACK_CONFIGURE_ENABLE=	large-stack
+
+BASH_CMD=	${LOCALBASE}/bin/bash
 
 OPTIONS_SUB=	yes
 

Modified: head/sysutils/e2fsprogs/Makefile
==============================================================================
--- head/sysutils/e2fsprogs/Makefile	Tue Dec  4 20:10:05 2018	(r486633)
+++ head/sysutils/e2fsprogs/Makefile	Tue Dec  4 22:36:34 2018	(r486634)
@@ -66,7 +66,8 @@ BASHTESTS_DESC=	Enable tests that require the GNU bash
 FUSEFS_CONFIGURE_ENABLE=	fuse2fs
 
 BUILD_DEPENDS+=	gdd:sysutils/coreutils
-BASHTEST_BUILD_DEPENDS=	bash:shells/bash
+BASHTESTS_BUILD_DEPENDS=	${BASH_CMD}:shells/bash
+BASH_CMD=	${LOCALBASE}/bin/bash
 
 LIB_DEPENDS+=	libblkid.so:misc/e2fsprogs-libblkid
 LIB_DEPENDS+=	libss.so:devel/e2fsprogs-libss
@@ -207,7 +208,7 @@ _checkaddargs+=TMPDIR=${WRKDIR}/tmp
 .endif
 
 .if ${PORT_OPTIONS:MBASHTESTS}
-_checkaddargs+=--eval SHELL:=${LOCALBASE}/bin/bash
+_checkaddargs+=--eval SHELL:=${BASH_CMD}
 .endif
 
 .if ${PORT_OPTIONS:MSLOWTESTS}



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