Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 23 Jun 2017 09:12:16 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Dirk Meyer <dinoex@FreeBSD.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r444162 - head/mail/fdm
Message-ID:  <20170623091216.GA59301@FreeBSD.org>
In-Reply-To: <201706230005.v5N05uAv003195@repo.freebsd.org>
References:  <201706230005.v5N05uAv003195@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--0F1p//8PRICkK4MW
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Fri, Jun 23, 2017 at 12:05:56AM +0000, Dirk Meyer wrote:
> New Revision: 444162
> URL: https://svnweb.freebsd.org/changeset/ports/444162
> 
> Log:
>   - fix build, add MAKE_JOBS_UNSAFE
>   PR:		220072
>   Submitted by:	Brad Sliger

Why not fix it properly instead, by adding missing dependency of `lex.c' (no
other file wants to include `parse.h') on it?  See attachment.

./danfe

--0F1p//8PRICkK4MW
Content-Type: text/x-diff; charset=us-ascii
Content-Disposition: attachment; filename="fdm.diff"

Index: Makefile
===================================================================
--- Makefile	(revision 444167)
+++ Makefile	(working copy)
@@ -13,8 +13,6 @@
 
 LIB_DEPENDS=	libtdb.so:databases/tdb
 
-MAKE_JOBS_UNSAFE=	yes
-
 USE_GITHUB=	yes
 GH_ACCOUNT=	nicm
 USES=		autoreconf:build ssl
@@ -34,6 +32,7 @@
 
 post-patch:
 	@${REINPLACE_CMD} -e 's|/etc|${PREFIX}/etc|g' ${WRKSRC}/*.[15] ${WRKSRC}/MANUAL
+	@${ECHO_CMD} 'lex.c: parse.h' >> ${WRKSRC}/Makefile.am
 
 pre-configure:
 	@cd ${WRKSRC} && sh autogen.sh

--0F1p//8PRICkK4MW--



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