svn commit: r438115 - head/security/bearssl

[Baptiste Daroussin]

On Sun, Apr 09, 2017 at 09:46:48PM +0200, Baptiste Daroussin wrote:
> On Sun, Apr 09, 2017 at 09:40:17PM +0200, Baptiste Daroussin wrote:
> > On Sun, Apr 09, 2017 at 07:26:28PM +0000, Mark Felder wrote:
> > > Author: feld
> > > Date: Sun Apr  9 19:26:28 2017
> > > New Revision: 438115
> > > URL: https://svnweb.freebsd.org/changeset/ports/438115
> > > 
> > > Log:
> > >   security/bearssl: Update to 0.4
> > >   
> > >   Now requires GCC. The Clang build is not functional as it fails to link.
> > >   The cause is under investigation.
> > > 
> > > Modified:
> > >   head/security/bearssl/Makefile
> > >   head/security/bearssl/distinfo
> > >   head/security/bearssl/pkg-plist
> > > 
> > > Modified: head/security/bearssl/Makefile
> > > ==============================================================================
> > > --- head/security/bearssl/Makefile	Sun Apr  9 18:29:32 2017	(r438114)
> > > +++ head/security/bearssl/Makefile	Sun Apr  9 19:26:28 2017	(r438115)
> > > @@ -2,8 +2,8 @@
> > >  # $FreeBSD$
> > >  
> > >  PORTNAME=	bearssl
> > > -PORTVERSION=	0.2
> > > -PORTREVISION=	1
> > > +PORTVERSION=	0.4
> > > +PORTREVISION=	0
> > >  CATEGORIES=	security devel
> > >  MASTER_SITES=	https://bearssl.org/
> > >  
> > > @@ -12,14 +12,12 @@ COMMENT=	Implementation of TLS/SSL in C
> > >  
> > >  LICENSE=	MIT
> > >  
> > > -USES=		compiler:c11
> > > -
> > > -post-patch:
> > > -	${REINPLACE_CMD} 's|CC = gcc|CC ?= gcc|' ${WRKSRC}/Makefile
> > > +USES=		compiler
> 
> There is no point in leaving that
> > > +USE_GCC=	yes
> > >  
> > >  do-install:
> > > -	${INSTALL_PROGRAM} ${WRKSRC}/brssl ${STAGEDIR}${LOCALBASE}/bin/
> > > -	(cd ${WRKSRC}/inc && ${COPYTREE_SHARE} . ${STAGEDIR}${LOCALBASE}/include/)
> > > -	${INSTALL_DATA} ${WRKSRC}/libbearssl.a ${STAGEDIR}${LOCALBASE}/lib/
> > > +	${INSTALL_PROGRAM} ${WRKSRC}/build/brssl ${STAGEDIR}${LOCALBASE}/bin/
> > > +	${INSTALL_DATA} ${WRKSRC}/build/libbearssl.so ${STAGEDIR}${LOCALBASE}/lib/
> 
> Should be INSTALL_LIB
> 
> > > +	(cd ${WRKSRC}/inc && ${COPYTREE_SHARE} \* ${STAGEDIR}${LOCALBASE}/include)
> > >  


https://people.freebsd.org/~bapt/bearssl.diff this builds fine for me

I don't know why you added compiler:c11 before but given I have no gcc 4.2
available I cannot test without it so I kept it

if you pass variables with args there is no need to replace VAR = by VAR ?= in
makefiles.

tested in 10.3, 11 and current

Bapt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-all/attachments/20170409/c467a61d/attachment.sig>