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

Re: r1578670 - Fix the order of node record headers written by svndumpfilter

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Mon, 14 Apr 2014 12:02:49 +0100

Julian Foad <julianfoad_at_btopenworld.com> writes:

> This commit changes the output format of paths from no leading slash
> ('relpath' style) to having a leading slash ('fspath' style). This
> seems to be against the desired path format for a dump file, although
> the documentation in <notes/dump-load-format.txt> is not explicit.
> I noticed this because it crashes  'svnrdump load':
> $ svnrdump load file://$PWD/rr < the-dump-filtered
> lt-svnrdump: subversion/libsvn_subr/dirent_uri.c:1256: svn_relpath_dirname: Assertion `relpath_is_canonical(relpath)' failed.
> Aborted (core dumped)
> That's probably a bug in itself; 'svnadmin load' accepts them.

I can't reproduce the SEGV with 1.6, 1.7, 1.8 or trunk.

The code at the top of new_node_record explicitly adds a leading '/' if
none is present so "node_path + 1" should be a safe way to print the
path without a leading '/'.

Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*
Received on 2014-04-14 13:03:28 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.