svn commit: r473756 - head/security/teleport

[Mathieu Arnold]

On Mon, Jul 02, 2018 at 09:21:20PM +0200, Mathieu Arnold wrote:
> On Mon, Jul 02, 2018 at 04:59:13PM +0000, Sean Chittenden wrote:
> > Author: seanc
> > Date: Mon Jul  2 16:59:13 2018
> > New Revision: 473756
> > URL: https://svnweb.freebsd.org/changeset/ports/473756
> > 
> > Log:
> >   Optimize teleport build further and reuse the same sed invocation.
> >   
> >   Submitted by:	mat
> >   Approved by:	mat (mentor, implicit)
> >   Pointy hat:	seanc
> > 
> > Modified:
> >   head/security/teleport/Makefile
> > 
> > Modified: head/security/teleport/Makefile
> > ==============================================================================
> > --- head/security/teleport/Makefile	Mon Jul  2 16:45:53 2018	(r473755)
> > +++ head/security/teleport/Makefile	Mon Jul  2 16:59:13 2018	(r473756)
> > @@ -45,9 +45,9 @@ post-extract:
> >  post-patch:
> >  	@${REINPLACE_CMD} -e 's|^GITREF=.*|GITREF=${GH_TAG_COMMIT}|' \
> >  		${WRKDIR}/${GO_TELEPORT_SRC_DIR}/version.mk
> > -	@${FIND} ${WRKDIR} -type f \
> > -		-exec ${SED} -i '' -e 's|\/var\/lib|/var/db|g' {} \; \
> > -		-exec ${SED} -i '' -e 's|\/usr\/bin\/hostname|/bin/hostname|g' {} \;
> > +	@${FIND} ${WRKDIR} -type f -exec ${SED} -i '' \
> > +		-e 's|\/var\/lib|/var/db|g' \
> > +		-e 's|\/usr\/bin\/hostname|/bin/hostname|g' {} \;
> 
> Now, you are only running one sed for each file in WRKDIR instead of
> two.  Let's cut that down to only one sed call for all the files, please
> replace "\;" with "+".

Ping?

-- 
Mathieu Arnold
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 963 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-head/attachments/20180709/664fa210/attachment.sig>