Date: Tue, 6 Oct 2015 19:51:35 +0300 From: Dmitry Marakasov <amdmi3@amdmi3.ru> To: Sunpoet Po-Chuan Hsieh <sunpoet@FreeBSD.org> Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r398474 - head/Mk/Uses Message-ID: <20151006165135.GB68979@hades.panopticon> In-Reply-To: <201510031706.t93H6qiu007585@repo.freebsd.org> References: <201510031706.t93H6qiu007585@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
* Sunpoet Po-Chuan Hsieh (sunpoet@FreeBSD.org) wrote:
> Log:
> - Add NO_PERL5_TEST: allow ports with USES=perl5 to run default do-test: defined in bsd.port.mk
> - Fix comment
>
> Modified:
> head/Mk/Uses/perl5.mk
>
> Modified: head/Mk/Uses/perl5.mk
> ==============================================================================
> --- head/Mk/Uses/perl5.mk Sat Oct 3 16:30:19 2015 (r398473)
> +++ head/Mk/Uses/perl5.mk Sat Oct 3 17:06:52 2015 (r398474)
> @@ -308,7 +308,7 @@ fix-perl-things:
> @${RM} -f ${STAGEDIR}${PREFIX}/lib/perl5/${PERL_VER}/${PERL_ARCH}/perllocal.pod* || :
> @${RMDIR} -p ${STAGEDIR}${PREFIX}/lib/perl5/${PERL_VER}/${PERL_ARCH} 2>/dev/null || :
>
> -.if !target(do-test)
> +.if !target(do-test) && !defined(NO_PERL5_TEST)
> TEST_TARGET?= test
> TEST_WRKSRC?= ${BUILD_WRKSRC}
> do-test:
> @@ -317,5 +317,5 @@ do-test:
> .elif ${USE_PERL5:Mconfigure}
> cd ${TEST_WRKSRC}/ && ${SETENV} ${TEST_ENV} ${MAKE_CMD} ${TEST_ARGS} ${TEST_TARGET}
> .endif # USE_PERL5:Mmodbuild*
> -.endif # regression-test
> +.endif # do-test
> .endif # defined(_POSTMKINCLUDED)
Please roll this back. We already have NO_TEST specifically for this
purpose. Also, I'd be really grateful if tests-related infrastructure
changes are reviewed first with me CCd.
--
Dmitry Marakasov . 55B5 0596 FF1E 8D84 5F56 9510 D35A 80DD F9D2 F77D
amdmi3@amdmi3.ru ..: jabber: amdmi3@jabber.ru http://amdmi3.ru
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20151006165135.GB68979>