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

Re: Moving some of our tools to "main" subversion

From: Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
Date: Tue, 26 Aug 2014 15:02:43 +0200

On Tue, Aug 26, 2014 at 2:11 PM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:

> On 19 August 2014 15:34, Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
> wrote:
> > Hi there,
> >
> > At the SHF hackathon, we talked what tools should be installed
> > by default and which should be part of tools.
> >
> > We decided to make
> >
> > * svn-bench (reported as useful in the field) and
> I agree that 'svn-bench' could be useful for users, but what is the
> reason for non-standard naming? Why this tool is not named 'svnbench'
> ?
>

This has already been brought up here:
http://svn.haxx.se/dev/archive-2014-08/0221.shtml

Personally, I'd be fine with renaming it. Others think
we shouldn't do it because of "released". Realistically,
we will break hardly anybody's setup by renaming it.

> * svnfsfs (disaster recovery tools should be available by default)
> >
> I doubt that 'svnfsfs' can be moved to "main" Subversion in the
> current state because it uses internal library headers and functions
> from libsvn_fs_fs:
> [[[
> #include "../libsvn_fs_fs/index.h"
> #include "../libsvn_fs_fs/pack.h"
> #include "../libsvn_fs_fs/rev_file.h"
> #include "../libsvn_fs_fs/util.h"
> ]]]
>
> Have you considered this?

Only to the degree that I mentioned that the tool links
against libsvn_fs_fs directly instead of using the loader.
People seemed fine with that.

> What is about ABI/API issues?

You have a point there. Those functions probably need
to be exported like any other private API that we use
(Windows DLL exports come to mind).

> Note that we
> never include such headers in current Subversion code except
> "fs-loader.h" and tests.
>

Would moving the declarations (2 structs, 10 functions)
to a new "include/private/svn_fs_fs_private.h" be sufficient
in your opinion?

-- Stefan^2.
Received on 2014-08-26 15:03:20 CEST

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.