svn commit: r473756 - head/security/teleport

Sean Chittenden seanc at FreeBSD.org
Sat Jul 14 14:25:23 UTC 2018


> > 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 "+".

[ Sorry, I've been traveling for the last week and unable to dig into this or test ]

Huh, I didn't know find(1) had an `-exec +` mode, and yes, that's the right way
to update the ports.  TIL.  Thanks!  -sc

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


More information about the svn-ports-all mailing list