On Wed, Dec 26, 2012 at 8:31 AM, Stefan Fuhrmann <
stefanfuhrmann_at_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.)
Received on 2012-12-26 16:20:56 CET