Re: svn commit: r1425778 - in /subversion/trunk: build.conf tools/dev/fsfs-access-map.c
On Wed, Dec 26, 2012 at 8:31 AM, Stefan Fuhrmann <
> 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
This is an archived mail posted to the Subversion Dev