Re: Issue #4476 - Mergeinfo containing r0 makes svnsync and svnadmin dump fail

2014-12-04 Thread Julian Foad
Julian Foad wrote: > http://subversion.tigris.org/issues/show_bug.cgi?id=4476 [...] > PROPOSAL > > In the presence of a mergeinfo property that refers to r0, at the > libsvn_repos > API: > > 'dump' >   shall dump the property verbatim (with nowarning). (Done.) > > 'load', with 'validate proper

Re: Issue #4476 - Mergeinfo containing r0 makes svnsync and svnadmin dump fail

2014-12-04 Thread Julian Foad
I (Julian Foad) wrote: >> I'll come to 'svnsync' later, but basically my current thought is it >> should 'correct' the mergeinfo by removing the r0 reference. > > I have written a test for this, and hacked up code to do this by textual > substitution. It isn't quite right -- for example it wo

Re: Issue #4476 - Mergeinfo containing r0 makes svnsync and svnadmin dump fail

2014-12-04 Thread Branko Čibej
On 04.12.2014 19:40, Julian Foad wrote: > I (Julian Foad) wrote: > >>> I'll come to 'svnsync' later, but basically my current thought is it >>> should 'correct' the mergeinfo by removing the r0 reference. >> I have written a test for this, and hacked up code to do this by textual >> substitutio

Re: Issue #4476 - Mergeinfo containing r0 makes svnsync and svnadmin dump fail

2014-12-04 Thread Julian Foad
[Second attempt to attach the patch.] I (Julian Foad) wrote: >> I'll come to 'svnsync' later, but basically my current thought is it >> should 'correct' the mergeinfo by removing the r0 reference. > > I have written a test for this, and hacked up code to do this by textual > substitution. It is

Re: svn commit: r1643139 - /subversion/trunk/subversion/libsvn_fs_fs/caching.c

2014-12-04 Thread Branko Čibej
On 05.12.2014 01:04, stef...@apache.org wrote: > Author: stefan2 > Date: Fri Dec 5 00:01:54 2014 > New Revision: 1643139 > > URL: http://svn.apache.org/viewvc?rev=1643139&view=rev > Log: > Fix the FSFS-specific txn root cleanup misbehavior demonstrated by r16. r16?

Re: svn commit: r1643139 - /subversion/trunk/subversion/libsvn_fs_fs/caching.c

2014-12-04 Thread Stefan Fuhrmann
On Fri, Dec 5, 2014 at 5:04 AM, Branko Čibej wrote: > On 05.12.2014 01:04, stef...@apache.org wrote: > > Author: stefan2 > > Date: Fri Dec 5 00:01:54 2014 > > New Revision: 1643139 > > > > URL: http://svn.apache.org/viewvc?rev=1643139&view=rev > > Log: > > Fix the FSFS-specific txn root cleanup