[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Fri, 5 Jul 2013 00:05:02 +0300

ocleirig_at_tigris.org wrote on Thu, Jul 04, 2013 at 13:43:08 -0700:
> ------- Additional comments from ocleirig_at_tigris.org Thu Jul 4 13:43:08 -0700 2013 -------
> Thanks for [cmpilato's] feedback.
>
> I was thinking that in modern SVN the author and commit date would always exist
> so that was a way to approach the check.
>
> But the core issue is that the 4389-filered.dump file contains revision
> properties but when it is imported they are not and no warning is given.
>
> The svnadmin load and svnrdump load commands do make some consistency checks
> (like verifiying the checksums) so it seemed to me that this is another
> variation to add.
>

+1.

From notes/dump-load-format.txt:

    Revision-number: <N>
    Prop-content-length: <P>
    Content-length: <L>
    ... <P> gives the length in bytes of the following property section.
    <L> gives the body length of the entire Revision record. These two
    numbers will be *identical* for a Revision record; ...
(emphasis in the original)

So it would be useful for the parser to set both headers when parsing
a Revision-number headers --- and to warn if one of them is absent, or
if they are set to different values.

Daniel

> I'd be happy with the check you suggest where if properties exist and the
> Prop-content-length header is missing to fail the import.
Received on 2013-07-04 23:05:39 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.