On Wed, Dec 26, 2012 at 8:31 AM, Stefan Fuhrmann < stefanfuhrm...@alice-dsl.de> wrote:
> On 12/26/2012 12:54 PM, Bert Huijben wrote: > > One of the build.conf changes applies to __ALL_TESTS__, not to libs. > (There is no such section). > > > I took that from the diff output "libs = __ALL__". > Corrected the log message now. > > > I’m not sure why we add tools that are not used by the test tools to the > __ALL_TESTS__ target? Do you intend to run it from the test suite. (On > which platforms?) > > > No. I don't expect that tool to be used in tests. > I only want to get it built. __ALL_TESTS__ seems > to be the target we use for this purpose already > (e.g. svn-rep-sharing-stats). > > Broken window? Maybe, we need a new "just > build it" target instead of misusing __ALL_TESTS__ > as the kitchen sink? > Possibly. It may also be useful to consider whether or not we should ship tools like this. (Though, I expect that packagers would probably them out into separate packages anyway, so maybe it's moot.)
