Re: [Tracker] [PATCHES] Various patches for tracker-0.5.3



2007/1/9, Saleem Abdulrasool <compnerd gentoo org>:


tracker-0.5.3-disable-tests.patch: As with the debug code paths, this
fixes the test issue.


I'd propose to dump the BUILD_TEST conditional completely and instead
use a builtin feature of automake:
The "check_" prefix.

Everything which is prefixed with "check_", (e.g. check_PROGRAMS) is
only built if you run "make check". It is also not installed, the
binaries/libraries etc. listed there are only for testing purposes.
This way you can easily setup a test suite, which checks the proper
functioning of tracker.
Imho it would be a good idea to write more test cases. IIRC "make
check" is implicitely run when you do "make distcheck", which should
always be used before a release.
This way we could easily check regressions.

Cheers,
Michael



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]