[RFC/PATCH 0/15] svn-fe: support for property deltas (but not text deltas yet)

Hi,

Jonathan Nieder wrote:

Another chapter. This one is mostly David's work. One can test it (I haven't tried) by taking out the safety

	die("text deltas not supported");

and letting it import from some real-world deltified dump file, treating deltas as the full text. Then one can validate as usual, by checking the modes of files in each revision of the imported tree.

I've done a little refactoring to more easily understand the patch, but the heart of the work is still David's. Any bugs and infelicities are probably mine (especially the evil hack to support switching between executables and symlinks).

Especially useful would be a nicer set of tests. A tool to create dump files based on a more concise description would make this and some future tasks a lot easier (hint, hint).

Thoughts? Improvements? Any of these ready to go? I can rearrange to put the uncontroversial patches at the beginning once it is clear what those are.

David Barr (1):
  vcs-svn: Implement Prop-delta handling

Jonathan Nieder (14):
  vcs-svn: Check for errors from open()
  vcs-svn: Eliminate node_ctx.srcRev global
  vcs-svn: Eliminate node_ctx.mark global
  vcs-svn: Unclutter handle_node by introducing have_props var
  vcs-svn: Use mark to indicate nodes with text
  vcs-svn: handle_node: Handle deletion case early
  vcs-svn: Replace = Delete + Add
  vcs-svn: Merge repo_replace and repo_modify functions
  vcs-svn: Delay read of per-path properties
  vcs-svn: Reject path nodes without Node-action
  vcs-svn: More dump format sanity checks
  vcs-svn: Make source easier to read on small screens
  vcs-svn: Split off function for handling of individual properties
  vcs-svn: Sharpen parsing of property lines

 contrib/svn-fe/svn-fe.c |    3 +-
 t/t9010-svn-fe.sh       |  301 ++++++++++++++++++++++++++++++++++++++++++++++-
 test-svn-fe.c           |    3 +-
 vcs-svn/repo_tree.c     |   21 +--
 vcs-svn/repo_tree.h     |    3 +-
 vcs-svn/svndump.c       |  191 +++++++++++++++++++-----------
 vcs-svn/svndump.h       |    2 +-
 7 files changed, 436 insertions(+), 88 deletions(-)