On Sat, Apr 12, 2014 at 06:22:31PM +0200, Henrik Friedrichsen wrote:
> Hello,
>
> This patch fixes a bug in which the lua library ssl.so is linked with ld
> instead of cc.
>
> This results in the stack-protector symbols being missing and therefore
> generates a useless library which cannot be used in lua.
>
> The patch sets the LD="cc" to fix this.
>
> For some reason this problem did not occur in -CURRENT until recently.
>
> OK?
I can confirm. Updated diff attached with REVISION and use of ${CC}.
Can I get an ok? Tested on amd64.
--
James Turner
Index: Makefile
===================================================================
RCS file: /cvs/ports/security/luasec/Makefile,v
retrieving revision 1.13
diff -u -p -u -p -r1.13 Makefile
--- Makefile 20 Mar 2014 19:21:00 -0000 1.13
+++ Makefile 12 Apr 2014 17:06:05 -0000
@@ -4,6 +4,7 @@ SHARED_ONLY= Yes
COMMENT= lua binding to OpenSSL to provide TLS/SSL communication
VERSION= 0.5
+REVISION= 0
DISTNAME= luasec-${VERSION}
CATEGORIES= security
MASTER_SITES= https://github.com/brunoos/luasec/archive/
@@ -20,7 +21,7 @@ NO_TEST= Yes
MODLUA_RUN_DEPENDS+= net/luasocket
WANTLIB+= crypto ssl
-MAKE_ENV+= INC_PATH="-I${MODLUA_INCL_DIR}" SAVECFLAGS="${CFLAGS}"
+MAKE_ENV+= INC_PATH="-I${MODLUA_INCL_DIR}" SAVECFLAGS="${CFLAGS}"
LD="${CC}"
ALL_TARGET= bsd
WRKDIST= ${WRKDIR}/luasec-luasec-${VERSION}
