Re: svn commit: r24648 - in trunk: . subversion/include subversion/include/private subversion/libsvn_client subversion/libsvn_fs_base subversion/libsvn_wc subversion/tests/libsvn_subr

On Thu, 19 Apr 2007, Joe Swatosh wrote:

> Hi Paul,
>
> On 4/18/07, pburba@tigris.org <pburba@tigris.org> wrote:
> >Author: pburba
> >Date: Wed Apr 18 18:01:05 2007
> >New Revision: 24648
> >
> >Log:
> >Move svn_mergeinfo__to_string to new svn-private header for mergeinfo APIs.
> >
>
> This caused a test failure in the Ruby bindings:
>
> test_merge_info_to_s(SvnCoreTest):
> NoMethodError: undefined method `mergeinfo__to_string' for Svn::Core:Module
> D:/SVN/src-trunk/subversion/bindings/swig/ruby/svn/core.rb:643:in `to_s'
> ./subversion\bindings\swig\ruby\test\test_core.rb:748:in
> `test_merge_info_to_s'
>
> Are the private functions intended to be available to the bindings
> implementations or only to subversion itself? If the bindings are
> supposed to be able to use them, we will probably have to create a
> proxy for this?

I think that the bindings should be able to use this routine
internally. Exposing, or at least publicizing it, outside of
Subversion is more dubious...