[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1425778 - in /subversion/trunk: build.conf tools/dev/fsfs-access-map.c

From: Hyrum K Wright <hyrum_at_hyrumwright.org>
Date: Wed, 26 Dec 2012 10:20:16 -0500

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

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.