cvs commit: ports/lang/s9fes Makefile

Alexey Dokuchaev danfe at FreeBSD.org
Wed Nov 16 04:44:11 UTC 2011


On Wed, Nov 16, 2011 at 03:21:34AM +0000, Eitan Adler wrote:
> eadler      2011-11-16 03:21:34 UTC
> 
>   Modified files:
>     lang/s9fes           Makefile 
>   Log:
>   - mark port unsafe with -j

Oh come on.  Why blindly mark ports as -jX unsafe when trivial fix is
discoverable in a minute?  Also, REINPLACE_CMD's usage is very lame, and
should be removed in favor of MAKE_ARGS.

Please consider attached patch.

./danfe
-------------- next part --------------
Index: Makefile
===================================================================
RCS file: /home/danfe/fbsd/FreeBSD-CVS/ports/lang/s9fes/Makefile,v
retrieving revision 1.16
diff -u -r1.16 Makefile
--- Makefile	9 Oct 2010 00:30:33 -0000	1.16
+++ Makefile	16 Nov 2011 04:42:26 -0000
@@ -13,13 +13,14 @@
 MAINTAINER=	nmh at t3x.org
 COMMENT=	A portable Scheme interpreter with a Unix interface
 
+MAKE_ARGS=	PREFIX=${PREFIX} LIBDIR=${DATADIR}
 WRKSRC=		${WRKDIR}/s9
 
 MAN1=		s9.1
 MANCOMPRESSED=	yes
 
 post-patch:
-	${REINPLACE_CMD} -e "s,PREFIX=.*,PREFIX=${PREFIX}," ${WRKSRC}/Makefile
-	${REINPLACE_CMD} -e "s,LIBDIR=.*,LIBDIR=${DATADIR}," ${WRKSRC}/Makefile
+# Fix parallel (-jX) build: add missing target prerequisite
+	@${REINPLACE_CMD} -e '/^arse-core.image/s,:,: s9,' ${WRKSRC}/Makefile
 
 .include <bsd.port.mk>


More information about the cvs-all mailing list