[PATCH] calling diff_summarize from SVN Perl bindings

[[[
    Make diff_summarize callable from Perl binding.

    [in subversion/bindings/swig]
    * svn_client.i
        Add typemap for (svn_client_diff_summarize_func_t, void*) arguments
sequence.
    * perl/libsvn_swig_perl/swigutil_pl.h
        Declare svn_swig_pl_thunk_client_diff_summarize_func as callback
thunk.
    * perl/libsvn_swig_perl/swigutil_pl.c
        Define svn_swig_pl_thunk_client_diff_summarize_func as callback
thunk.
    * perl/native/Client.pm
        Add document for diff_summarize method;
        Expose svn_client_diff_summarize_kind_t as
SVN::Client::Summarize::*.
   ]]]

On Sat, Dec 27, 2008 at 6:06 PM, Daniel Shahaf <d.s_at_daniel.shahaf.name>wrote:

> James Fang wrote on Thu, 27 Nov 2008 at 16:49 +0800:
> > at least in Perl, the SVN::Client::diff_summarize can not be invoke( it
> > reports illegal argument error).
> > i find the problem lies in that the "svn_client_diff_summarize_func_t"
> type
> > lacks the corresponding
> > SWIG declaration( compared with "svn_log_message_receiver")
>
> ...
>
> > since i am new to SWIG and Perl, i can not get to a ideal solution yet. I
> > just manual edited the
> > swig/perl/native/svn_client.c,
> > swig/perl/native/Core.pm,
> > swig/libsvn_swig_perl/swigutil_pl.c to get it worked.
> > But this is obvious a temporary solution, is there any plan for a more
> > complete language bindings?
> >
>
> I don't know anyone specifically working on that at the moment. However,
> since you already did the work to support those APIs, could you repackage
> it as a patch and send it to dev@? (Please read [1] before doing so.)
>
> Thanks,
>
> Daniel
>
> [1] http://subversion.tigris.org/hacking.html#patches
>
> >
> >
>

-- 
James Fang