Plugging ATF tests into the build and other cleanups

Simon J. Gerraty sjg at
Tue Oct 29 21:22:40 UTC 2013

On Sun, 27 Oct 2013 18:12:12 -0400, Julio Merino writes:
>The one concern I have here is having to keep track of all tests in
>tools/build/mk/ so that setting
>WITHOUT_TESTS=no cleans up /usr/tests. This will be a pain to maintain


>and a sure source of inconsistencies. If we could special-case this to
>make it more automatic, do you have any suggestions?

Iff ATF/Kyua were the only thing populating /usr/tests/ you could just
rm -rf that dir, but that's not how delete-old-* works else

KYUAFILES!= cd $DESTDIR/ && find usr/tests -name Kyuafile

would be all you'd need.
You can't even do OLD_FILES+= ${KYUAFILES:H}/* 

It could be that a new delete-old- target might be useful,

>As usual:


looks ok - assuming folk are ok with s,ATF,TESTS,


I'm not so crazy about.  
What's wrong with requiring folk to set TESTS_SUBDIR[S] ?


I'm also not crazy about this one (the change to lib/Makefile)
What is the expected use? and why can't everything that is needed be
done within the context of the individual lib/*/tests/ dir?
That's how we are doing it.


FWIW I prefer ${.CURDIR:H:H} etc rather than ${.CURDIR}/../..
makes for much neater paths (also bit quicker on nfs)

It would be nice to be able to rely on SRCTOP being correctly set
(easy with bmake, have to think about fmake...)

ATF_SRC = ${SRCTOP}/contrib/atf

could then be set on one place
but otherwise ok.

More information about the freebsd-testing mailing list