Re: Issue 3895, rejecting invalid svn:mergeinfo on commit

2011-07-11 Thread Stefan Sperling
On Mon, Jul 11, 2011 at 02:13:46PM +0100, Philip Martin wrote: > Stefan Sperling writes: > > > We should allow invalid mergeinfo to exit the repository, e.g. in a dump > > file. But input paths, such as svnadmin load, should refuse invalid > > mergeinfo. > > We have a similar problem with inval

Re: Issue 3895, rejecting invalid svn:mergeinfo on commit

2011-07-11 Thread Philip Martin
Stefan Sperling writes: > We should allow invalid mergeinfo to exit the repository, e.g. in a dump > file. But input paths, such as svnadmin load, should refuse invalid mergeinfo. We have a similar problem with invalid svn:log (non-LF line-ends). In that case load fails with an error but sync c

Re: Issue 3895, rejecting invalid svn:mergeinfo on commit

2011-07-11 Thread Stefan Sperling
On Mon, Jul 11, 2011 at 11:33:57AM +0100, Philip Martin wrote: > http://subversion.tigris.org/issues/show_bug.cgi?id=3895 > > This issue is about rejecting commits that attempt to introduce invalid, > i.e. fails to parse, svn:mergeinfo. This has been fixed on trunk and > was backported to 1.6.17.

Issue 3895, rejecting invalid svn:mergeinfo on commit

2011-07-11 Thread Philip Martin
http://subversion.tigris.org/issues/show_bug.cgi?id=3895 This issue is about rejecting commits that attempt to introduce invalid, i.e. fails to parse, svn:mergeinfo. This has been fixed on trunk and was backported to 1.6.17. It turns out that the fix was incomplete: it didn't fix commits through