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

Re: [PATCH v2] Add svnrdump

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sat, 24 Jul 2010 10:18:33 +0300

Ramkumar Ramachandra wrote on Sat, Jul 24, 2010 at 03:37:06 +0530:
> 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?
>

Revving these interfaces falls under the "editor v2" header, for which see
svn_editor.h (not implemented yet) and notes/editor-v2.txt.

BTW: adding anything to those interfaces is a LOT of work. (You are most
welcome to tackle it, of course, but don't think it's a job for an afternoon.)

> -- Ram
Received on 2010-07-24 09:20:12 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.