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

Re: svn:externals parser API

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Wed, 22 Feb 2012 15:41:31 +0200

Nikolaus Demmel wrote on Wed, Feb 22, 2012 at 14:22:26 +0100:
>
> Am 22.02.2012 um 11:16 schrieb Daniel Shahaf:
>
> > Daniel Shahaf wrote on Wed, Feb 22, 2012 at 11:31:00 +0200:
> >> Stefan Sperling wrote on Wed, Feb 22, 2012 at 00:08:35 +0100:
> >>> On Tue, Feb 21, 2012 at 11:50:54PM +0100, Stefan Sperling wrote:
> >>>> I think svn_wc_parse_externals_description3() should do what you need.
> >>>> Does this help you?
> >>>>
> >>>> The function is mapped in the python bindings:
> >>>>
> >>>>>>> import svn.wc
> >>>>>>> svn.wc.svn_wc_parse_externals_description3
> >>>> <function svn_wc_parse_externals_description3 at 0x20779a758>
> >>>
> >>>
> >>
> >> Based the difference between the signatures of the
> >> svn_wc_parse_externals_description() and
> >> svn_wc_parse_externals_description2(), I would suspect that someone
> >> needs to provide a Python implementation of the following typemap:
> >>
> >> #ifdef SWIGRUBY
> >> %typemap(argout) apr_array_header_t **externals_p {
> >> %append_output(svn_swig_rb_apr_array_to_array_external_item2(*$1));
> >> }
> >> #endif
> >>
> >
> > Done in r1292223:
> >
> > % PYTHONPATH=$prefix/svn-t1/lib/svn-python/ python -c \
> > 'import svn,svn.wc; print svn.wc.svn_wc_parse_externals_description3("","",1)'
> > []
> > %
> >
> > It will be available in 1.8.0.
> >
> > (perhaps we should relax our compat guidelines to allow adding exposed
> > APIs to the bindings in patch releases?)
> >
>
> Hi,
>
> this sound pretty good, thanks for the quick replies. If I understand this correctly (I havent used SWIG before), we could get this in the Perl bindings by adding a similar typemap for Perl?
>

Sounds right. (I haven't checked whether something beyond a typemap
would be needed for Perl.)

> I won't attempt it myself, since I know neither Perl, nor swig, but I will head over to the git folks and tell them that this should be easy to add.
>

Sure.

http://subversion.apache.org/patches
http://github.com/apache/subversion

:-)

> Cheers,
> Nikolaus
> (please CC on answers, I'm not subscribed to the list - thanks :-) )
Received on 2012-02-22 14:43:47 CET

This is an archived mail posted to the Subversion Users mailing list.