svn commit: r384307 - head/math/coinmp

Alexey Dokuchaev danfe at FreeBSD.org
Sun Apr 19 17:34:44 UTC 2015


On Sun, Apr 19, 2015 at 05:16:58PM +0000, Kurt Jaeger wrote:
> New Revision: 384307
> URL: https://svnweb.freebsd.org/changeset/ports/384307
> 
> Log:
>   math/coinmp: change comma, target for check/test etc as suggested by danfe
>   
>   PR:		199502
>   Submitted by:	danfe

Wrong attribution; the PR was not submitted by me.

> [...]
>  check regression-test test: build
> -	@(cd ${WRKSRC}; ${SETENV} ${MAKE_ENV} ${MAKE} ${_MAKE_JOBS} \
> -	${MAKE_ARGS} test)
> +	${SETENV} ${MAKE_ENV} ${MAKE} ${_MAKE_JOBS} \
> +	${MAKE_ARGS} test -C ${WRKSRC}

Wrong indentation.  I'd appreciate if you'd waited for my final review,
given that I've suggested these improvements.

This is partially my fault since I didn't explain Carlos that PR was not
necessary for this at all clearly enough. :(

./danfe


More information about the svn-ports-all mailing list