search

cstroe / svndumpapi

A Java library for manipulating a Subversion dump file.
GNU Affero General Public License v3.0
5 stars 1 forks source link

Fix svn:log property to be compatible with newer versions of Subversion #39

Closed cstroe closed 9 years ago

cstroe commented 9 years ago

If you try to rebuild a SVN 1.6 repo with an svnadmin command from a newer version of SVN (in my case 1.8.8), you may encounter the following:

svnadmin: E125005: Invalid property value found in dumpstream; consider repairing the source or using --bypass-prop-validation while loading.
svnadmin: E125005: Cannot accept non-LF line endings in 'svn:log' property

So it seems like in Subversion 1.6 you could have a svn:log property that didn't end in a line feed character, but in Subversion 1.8 that's no longer allowed.

It seems like a trivial exercise to make a property mutator that will guarantee that all svn:log properties end with a line feed character. Maybe do this?

cstroe commented 9 years ago

So it turns out that part of the problem is that an svn:log property contains windows style line feeds (\r\n), mixed in with unix style (\n). I think converting the line feeds to unix style should be a good fix for this issue.