[PATCH] Remove redundant url-encoding added in r917523

2011-03-24 Thread vijay
Hi, This patch adds a testcase for the regression triggered by r917523 and fixes it. The revision r917523 do some url-encodings to the paths and uris in subversion/mod_dav_svn/mirror.c. $svn log -r917523 r917523 | kam

Re: [PATCH] Remove redundant url-encoding added in r917523

2011-03-24 Thread Kamesh Jayachandran
Thanks for the Patch Vijay. In my FC14 your testcase passes both with and without patch. I am investigating Will get back. With regards Kamesh Jayachandran On 03/24/2011 04:11 PM, vijay wrote: Hi, This patch adds a testcase for the regression triggered by r917523 and fixes it. The rev

Re: [PATCH] Remove redundant url-encoding added in r917523

2011-03-24 Thread vijay
Just now I came to know that it fails in neon only. I configured neon as a default ra_layer in my runtime configuration area. When I use serf as ra_layer, the commit succeeds. Thanks & Regards, vijayaguru On Thursday 24 March 2011 05:12 PM, Kamesh Jayachandran wrote: Thanks for the Patch Vija

Re: [PATCH] Remove redundant url-encoding added in r917523

2011-03-24 Thread Kamesh Jayachandran
On 03/24/2011 05:54 PM, vijay wrote: Just now I came to know that it fails in neon only. I configured neon as a default ra_layer in my runtime configuration area. When I use serf as ra_layer, the commit succeeds. Yes that answers the failure I observed. Please provide a testcase that exhibit

Re: [PATCH] Remove redundant url-encoding added in r917523

2011-03-24 Thread Peter Samuelson
> > DAV svn > SVNParentPath /repositories > > > DAV svn > SVNParentPath /repositories-slave > SVNMasterURI "http://localhost/svn 1" > That seems like an incorrect configuration. The SVNMasterURI, like any other URI, should not have an unescaped space in it. I don't know if that ch

Re: [PATCH] Remove redundant url-encoding added in r917523

2011-03-24 Thread vijay
On Thursday 24 March 2011 10:09 PM, Peter Samuelson wrote: DAV svn SVNParentPath /repositories DAV svn SVNParentPath /repositories-slave SVNMasterURI "http://localhost/svn 1" That seems like an incorrect configuration. The SVNMasterURI, like any other URI, should not have an

Re: [PATCH] Remove redundant url-encoding added in r917523

2011-04-03 Thread vijay
On Thursday 24 March 2011 07:41 PM, Kamesh Jayachandran wrote: On 03/24/2011 05:54 PM, vijay wrote: Just now I came to know that it fails in neon only. I configured neon as a default ra_layer in my runtime configuration area. When I use serf as ra_layer, the commit succeeds. Yes that answe

Re: [PATCH] Remove redundant url-encoding added in r917523

2011-04-04 Thread Kamesh Jayachandran
Thanks Vijay for the detailed summary and the fix. I committed it at r1088602. With regards Kamesh Jayachandran On 04/04/2011 12:00 PM, vijay wrote: On Thursday 24 March 2011 07:41 PM, Kamesh Jayachandran wrote: On 03/24/2011 05:54 PM, vijay wrote: Just now I came to know that it fails in neo