Nice.  I don't see why svndumpfilter included r66, it might be a bug
(have you tried 1.7.6?).  We could also make the "This is an empty
revision for padding" information available as a revprop --- patches
welcome... (to svndumpfilter/main.c)

David Newman wrote on Mon, Sep 24, 2012 at 09:58:53 -0400:
>  On Fri, Sep 14, 2012 at 6:49 PM, Daniel Shahaf <d...@daniel.shahaf.name>
>  wrote:
> 
> > Can you post the actual parts of the dumpfile for those empty revisions?
> > (between successive "Revision-number: " lines)
> >
> >
> Sure.  Posted below are a couple of revisions.  I actually worked around
> this problem by writing a perl script that uses SVN::Dumpfile to read and
> write each revision but ignoring revisions with the svn:log property of
> "This is an empty revision for padding."  The fail case here would be if a
> developer commited a revision with that log message, but shame on them.
> 
> == Perl script ==
> #!/usr/bin/perl
> use SVN::Dumpfile;
> use strict;
> 
> my $olddf = SVN::Dumpfile->open(file => "original.dmp");
> my $newdf = $olddf->copy->create(file => "new.dump");
> 
> while ( my $node = $olddf->read_node ) {
>   if ($node->property('svn:log') ne 'This is an empty revision for
> padding.') {
>     $newdf->write_node($node);
>   }
> }
> 
> 
> == Dump File Sample ==
> 
> Revision-number: 66
> Prop-content-length: 112
> Content-length: 112
> 
> K 7
> svn:log
> V 38
> This is an empty revision for padding.
> K 8
> svn:date
> V 27
> 2009-06-18T13:13:48.000000Z
> PROPS-END
> 
> Revision-number: 67
> Prop-content-length: 112
> Content-length: 112
> 
> K 7
> svn:log
> V 38
> This is an empty revision for padding.
> K 8
> svn:date
> V 27
> 2009-06-23T12:38:37.000000Z
> PROPS-END

Reply via email to