Bug 182894 - www/mod_security has the wrong LIB_DEPENDS entry for lua
Summary: www/mod_security has the wrong LIB_DEPENDS entry for lua
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Pawel Pekala
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-11 05:30 UTC by xenophon+fbsdports
Modified: 2014-02-04 15:50 UTC (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description xenophon+fbsdports 2013-10-11 05:30:00 UTC
After enabling the LUA option in the ports' config, the www/mod_security port explicitly sets LIB_DEPENDS to pull in lang/lua.  However, on my system the dependency explicitly states version 5.1.5 of the lua library, when the library version number is 5.1.1.  (I don't know why there is a discrepancy.)  If I replace the lua LIB_DEPENDS entry in www/mod_security/Makefile with LUA_COMPS=lua:lib, the port builds and links against lua properly, as ports/Mk/bsd.lua.mk sets LIB_DEPENDS for the port.

Fix: Apply the following patch:



.if ${PORT_OPTIONS:MLUA}
 USE_LUA=       5.1+
+LUA_COMPS=     lua:lib
 CONFIGURE_ARGS+=       --with-lua=${LOCALBASE}
-LIB_DEPENDS+=  lua-5.1.5:${PORTSDIR}/lang/lua
 .else
 CONFIGURE_ARGS+=       --without-lua
 .endif--ymYWDS3cthrGVg6Z5bQSD0Qvmqn7Tf9bpDDakagAzgZ76isq
Content-Type: text/plain; name="file.diff"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment; filename="file.diff"

--- www/mod_security/Makefile.orig      2013-10-11 00:06:21.502494975 -0400
+++ www/mod_security/Makefile   2013-10-11 00:09:03.160494716 -0400
@@ -50,8 +50,8 @@
How-To-Repeat: (cd www/apache22 && make install BATCH=yes)
(cd www/mod_security && make install WITH=LUA)
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-10-11 05:30:24 UTC
Responsible Changed
From-To: freebsd-ports-bugs->araujo

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Marcelo Araujo freebsd_committer freebsd_triage 2013-11-29 08:20:38 UTC
Responsible Changed
From-To: araujo->freebsd-ports-bugs

Back to the pool.
Comment 3 Pawel Pekala freebsd_committer freebsd_triage 2014-02-03 21:10:08 UTC
Responsible Changed
From-To: freebsd-ports-bugs->pawel

I'll take it.
Comment 4 dfilter service freebsd_committer freebsd_triage 2014-02-04 15:40:47 UTC
Author: pawel
Date: Tue Feb  4 15:40:39 2014
New Revision: 342545
URL: http://svnweb.freebsd.org/changeset/ports/342545
QAT: https://qat.redports.org/buildarchive/r342545/

Log:
  - Fix build when LUA=on [1]
  - Remove leading article from COMMENT
  - Convert to LIB_DEPENDS format
  - Use option helpers
  - Support stage
  
  PR:		ports/182894 [1]
  Submitted by:	Matthew X. Economou <xenophon+fbsdports@irtnog.org>

Modified:
  head/www/mod_security/Makefile

Modified: head/www/mod_security/Makefile
==============================================================================
--- head/www/mod_security/Makefile	Tue Feb  4 14:53:26 2014	(r342544)
+++ head/www/mod_security/Makefile	Tue Feb  4 15:40:39 2014	(r342545)
@@ -9,12 +9,12 @@ PKGNAMEPREFIX=	${APACHE_PKGNAMEPREFIX}
 DISTNAME=	${PORTNAME:S/_//:S/2//}-apache_${PORTVERSION}
 
 MAINTAINER=	ports@FreeBSD.org
-COMMENT=	An intrusion detection and prevention engine
+COMMENT=	Intrusion detection and prevention engine
 
 LICENSE=	APACHE20
 
-LIB_DEPENDS+=	pcre:${PORTSDIR}/devel/pcre \
-		apr-1:${PORTSDIR}/devel/apr1
+LIB_DEPENDS+=	libpcre.so:${PORTSDIR}/devel/pcre \
+		libapr-1.so:${PORTSDIR}/devel/apr1
 
 USE_APACHE=	22+
 GNU_CONFIGURE=	yes
@@ -35,26 +35,18 @@ PLIST_FILES=	etc/modsecurity.conf-exampl
 		lib/mod_security2.so
 
 OPTIONS_DEFINE=	LUA MLOGC
-MLOGC_DESC=	Build ModSecurity Log Collector
 
-NO_STAGE=	yes
-.include <bsd.port.pre.mk>
+LUA_CONFIGURE_ON=	--with-lua=${LOCALBASE}
+LUA_CONFIGURE_OFF+=	--without-lua
+LUA_USE=		LUA=5.1+
+
+MLOGC_DESC=		Build ModSecurity Log Collector
+MLOGC_CONFIGURE_ON=	--with-curl=${LOCALBASE} --disable-errors
+MLOGC_CONFIGURE_OFF=	--disable-mlogc
+MLOGC_LIB_DEPENDS=	libcurl.so:${PORTSDIR}/ftp/curl
+MLOGC_PLIST_FILES=	bin/mlogc bin/mlogc-batch-load.pl
 
-.if ${PORT_OPTIONS:MMLOGC}
-LIB_DEPENDS+=	curl:${PORTSDIR}/ftp/curl
-CONFIGURE_ARGS+=	--with-curl=${LOCALBASE} --disable-errors
-PLIST_FILES+=	bin/mlogc bin/mlogc-batch-load.pl
-.else
-CONFIGURE_ARGS+=	--disable-mlogc
-.endif
-
-.if ${PORT_OPTIONS:MLUA}
-USE_LUA=	5.1+
-CONFIGURE_ARGS+=	--with-lua=${LOCALBASE}
-LIB_DEPENDS+=	lua-5.1.5:${PORTSDIR}/lang/lua
-.else
-CONFIGURE_ARGS+=	--without-lua
-.endif
+.include <bsd.port.options.mk>
 
 REINPLACE_ARGS=	-i ""
 AP_EXTRAS+=	-DWITH_LIBXML2
@@ -66,14 +58,16 @@ post-build:
 	cd ${WRKSRC} && ${SETENV} ${MAKE_ENV} ${MAKE} ${MAKE_FLAGS} ${MAKEFILE} ${MAKE_ARGS} mlogc
 .endif
 
+pre-install:
+	@${MKDIR} ${STAGEDIR}${PREFIX}/${APACHEMODDIR}
+
 post-install:
-	${INSTALL_DATA} ${WRKSRC}/modsecurity.conf-recommended ${PREFIX}/etc/modsecurity.conf-example
-.if !defined(NOPORTDOCS)
-	@${MKDIR} ${DOCSDIR}
-	@(cd ${WRKSRC} && ${COPYTREE_SHARE} "doc" ${DOCSDIR}/)
-.endif
+	${INSTALL_DATA} ${WRKSRC}/modsecurity.conf-recommended \
+		${STAGEDIR}${PREFIX}/etc/modsecurity.conf-example
+	@${MKDIR} ${STAGEDIR}${DOCSDIR}
+	(cd ${WRKSRC} && ${COPYTREE_SHARE} "doc" ${STAGEDIR}${DOCSDIR})
 .if ${PORT_OPTIONS:MMLOGC}
-	${INSTALL_PROGRAM} ${WRKSRC}/mlogc/mlogc ${PREFIX}/bin/
+	${INSTALL_PROGRAM} ${WRKSRC}/mlogc/mlogc ${STAGEDIR}${PREFIX}/bin
 .endif
 
-.include <bsd.port.post.mk>
+.include <bsd.port.mk>
_______________________________________________
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"
Comment 5 Pawel Pekala freebsd_committer freebsd_triage 2014-02-04 15:40:52 UTC
State Changed
From-To: open->closed

Committed. Thanks!