Skip to content

SVN loader: Normalize line endings when svn:eol-style property is set

Antoine Lambert requested to merge generated-differential-D407-source into generated-differential-D407-target

That diff tries to resolve the loading of a Subversion repository where wrong end of line style for a content has been stored in it.

For instance, it exists some corrupted svn dump files where a content has CRLF line endings while the svn:eol-style is set to native. According to the subversion documentation, this should not be possible as the content should have been saved internally by Subversion with LF line endings.

In a same manner, it also exists dump files storing a content with mixed LF and CRLF line endings while having the svn:eol-style set to native, which is also a violation of the Subversion specification.

Related #570 (closed)

Test Plan

To do, apart generating a svn dump file containing the corner cases described above I do not see any other way to test that fix.

Migrated from D407 (view on Phabricator)

Merge request reports