Julian Foad <julianf...@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*
