Re: [PATCH v2] Add svnrdump

Hi Stefan,

Stefan Sperling writes:
> I'd say get the basics right first, before worrying about features
> like this. In my opinion that means that it needs to produce the same
> output as svnadmin dump does. (I'm planning to revert r965892 soon,
> so you don't need to worry about the Prop-content-m5 and Prop-content-sha1
> fields.)

I agree. On this issue, the most urgent header required is the SHA1
checksum. First problem is that we have to patch the server to send it
along with the MD5 checksum. Second problem is that exposing it via
one extra argument in apply_textdelta and close_file will break
backwards compatibility. Any thoughts?

-- Ram
Received on 2010-07-24 00:09:26 CEST