Re: Authz on Collection of Repositories (was: Expansion of authz policy name leak)

2012-10-23 Thread roderich.sch...@gmail.com
> > I'm working on the patch to list only readable repositories. There is > already TODO comment in the code by cmpilato: > subversion\mod_dav_svn\repos.c:3461 > Please keep in mind that the problem is not restricted to parent-path collections of repositories: Since SVN 1.7 any user can "list

[PATCH] Small fixes to the Perl bindings

2013-01-31 Thread roderich.sch...@gmail.com
Hi, while trying to use svn_wc_parse_externals_description3 from Perl I stumbled over a fex things. Patches below are against trunk. perl-bindings-1.patch [[[ Make svn_wc_parse_externals_description3 available from Perl bindings. * subversion/bindings/swig/include/svn_containers.swg: Add out

Re: [PATCH] Small fixes to the Perl bindings

2013-02-08 Thread roderich.sch...@gmail.com
On Friday, February 8, 2013 1:53:03 AM UTC+1, Ben Reser wrote: > > For future reference though if you're adding support for a function > it'd be nice to add tests for it. > I'll keep that in mind. Thanks for applying, Ben. Cheers, Roderich

Re: Problem with Perl bindings?

2013-07-26 Thread roderich.sch...@gmail.com
On Friday, July 26, 2013 12:50:10 PM UTC+2, Philip Martin wrote: > > The trunk change doesn't merge cleanly to either 1.8 or 1.7 so you will > need to prepare backports: make backport branches from 1.7.x and 1.8.x > branches, merge the change, resolve the conflicts and then nominate the > bran

Re: Branch 'invoke-diff-cmd-feature' is ready for half-way review

2013-10-16 Thread roderich.sch...@gmail.com
On Wednesday, October 16, 2013 3:02:57 AM UTC+2, Gabriela Gibson wrote: > > The BRANCH-README file is here: > Substitutions: ;f1 original file ;f2 changed file "svn help diff" uses "old" and "new" (instead of "original" and "changed") to refer to t

Re: Branch 'invoke-diff-cmd-feature' is ready for half-way review

2013-10-16 Thread roderich.sch...@gmail.com
Looking over __create_custom_diff_cmd() in ./subversion/libsvn_subr/io.c: (1) The function doesn't strip double quotes, e.g. _create_custom_diff_cmd(..., /* cmd= */ "duff \"quoted\"", pool) will return the array { "duff", "\"quoted\"", NULL } You probably need to split cmd into tokens yourself

Re: [PATCH] Perl bindings corrupt "{DATE}" revision variable

2016-11-02 Thread roderich.sch...@gmail.com
On Wednesday, November 2, 2016 at 7:48:04 PM UTC+1, Daniel Shahaf wrote: > > Do the perl bindings use C89, like libsvn*? C89 forbids middle-of-block > declarations. > ... > The two maybe_croak() calls after this line (one of them in the patch > context, one just below it) would now print two r