[SVN-4389] svnrdump load should complain if revisions are missing svn:author, svn:date and svn:log properties

• Type: Improvement
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.7.x
• Fix Version/s: ---
• Component/s: svnrdump
• Assignee: Unassigned
• Reporter: Subversion Importer
• Created: 03/Jul/13 20:11
• Updated: 04/Jul/13 21:07

Description

I wrote an SVN Dump file parser to allow me to reconstruct lost history by 
inserting copyfrom data into the file and directory node properties of the dump 
file being processed.

i.e my-svndump-parser copyfrom-details.dat < rXYZ.dump > rXYZ-filtered.dump
 
My program had a bug which caused the Prop-content-length line to not be emitted 
for the revision node of the filtered dump file.

The svnrdump load bug is that without the Prop-content-length line present (the 
Content-length line was present) it silently discarded the revision properties 
without any warning or notice.

I think that svndump load should warn when loading a revision that does not have 
these basic revision properties: svn:author, svn:date, svn:log.

We fixed our issue by extracting the revision props from the source dump file 
and applying them separately using a script with many svn propset calls (with 
the pre-revprop-change hook enabled).

Original issue reported by ocleirig