hi, sorry to interrupt.
can you apply this patch so that it may come up in the next subversion
because one of our system rely on the diff_summarize from Perl. we do not
apply the patch ourself and build our own RPM packages when there are new
this patch is against the 1.5.4 release, and i have tested it on CentOS
On Thu, Jan 15, 2009 at 6:00 PM, James Fang <june.fang_at_gmail.com> wrote:
> 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
> * perl/libsvn_swig_perl/swigutil_pl.h
> Declare svn_swig_pl_thunk_client_diff_summarize_func as callback
> * perl/libsvn_swig_perl/swigutil_pl.c
> Define svn_swig_pl_thunk_client_diff_summarize_func as callback
> * perl/native/Client.pm
> Add document for diff_summarize method;
> Expose svn_client_diff_summarize_kind_t as
> 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(
>> > reports illegal argument error).
>> > i find the problem lies in that the "svn_client_diff_summarize_func_t"
>> > 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.
>> > 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  before doing so.)
>>  http://subversion.tigris.org/hacking.html#patches
> James Fang
Received on 2009-01-18 14:01:06 CET