svn commit: r296970 - head/lib/libxo/tests

Simon J. Gerraty sjg at juniper.net
Thu Mar 17 04:05:04 UTC 2016


NGie Cooper <yaneurabeya at gmail.com> wrote:
> > SUBDIR+=    encoder
> > 
> > +.if ${MACHINE} == "host"
> > +# make it easy to test without install
> > +TESTSDIR=    ${.OBJDIR}
> > +CFLAGS+=    -I${.CURDIR:H}
> > +.endif
> > +
> > .include <bsd.test.mk>
> 
> This commit seems unnecessary -- have you tried make check yet?

That appears to require installing things before you test that they
work - seems a bit backwards?

Note the above is only relevant when building with DIRDEPS.
Where we want to be able to build and run relevant tests as part of the
build as we go - when MACHINE = "host" that is.

Phil wants to fix this so that it runs all his test suite properly and
also need to arrange to auto-update the makefile here as new tests are
added to the distribution.
Probably need somthing like bmake's after-import target.

Anyway, being able to have this work in context of DIRDEPS_BUILD will
help (I haven't added the MK_TESTS hooks yet).



More information about the svn-src-head mailing list