Refactoring of *.test.mk

Julio Merino julio at meroh.net
Tue Oct 15 13:54:42 UTC 2013


On Tue, Oct 15, 2013 at 2:21 AM, Simon J. Gerraty <sjg at juniper.net> wrote:
> Thanks Julio,
>
> looks good.

Thanks for the reviews Simon and Rui.

> One nit; in hook-atf-run.diff and hook-kyua.diff I see:
[...]
> ATF_BUILD_CC?= ${DESTDIR}/usr/bin/cc
> ..
> TESTS_ENV+= ATF_BUILD_CC=${ATF_BUILD_CC}
> ..

OK, I've done this. And, at the same time, improved the way TESTS_ENV
is constructed by leaving the atf-specific bits in atf.test.mk and
moving the path handling to bsd.test.mk. PTAL.

I suspect that with 10 branched, we don't need approval from re@ for
submission at this point; is that correct?

That said, I think it's worth to merge this into 10 (now or after
release, doesn't matter much). Given that this touches yet-unused
files, it shouldn't be a major concern for re at . The reason for merging
into 10 is that it'd be nice if any future tests written in HEAD for
regressions encountered in 10 could be pulled up into the branch with
ease along the actual fixes.

Cheers,

-- 
Julio Merino / @jmmv


More information about the freebsd-testing mailing list