Re: Moving some of our tools to "main" subversion
On 27 August 2014 19:42, Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com> wrote:
> On Tue, Aug 26, 2014 at 4:00 PM, C. Michael Pilato <cmpilato_at_collab.net>
>> On 08/26/2014 09:02 AM, Stefan Fuhrmann wrote:
>> > 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?
>> I should think that would be sufficient. But then, it wasn't my opinion
>> that you solicited. :-)
> Implemented in r1620909.
This is completely wrong approach. Please revert immediately.
Proper way is to implement three specific semi-private functions in
libsvn_fs_fs for collecting stats, dumping and loading FS indexes and
then use in svnfsfs.
Received on 2014-08-27 17:55:28 CEST
This is an archived mail posted to the Subversion Dev