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

Re: svn commit: r1574868 - in /subversion/trunk/subversion: libsvn_repos/dump.c tests/libsvn_repos/repos-test.c

From: Branko Čibej <brane_at_wandisco.com>
Date: Sat, 08 Mar 2014 14:14:48 +0100

On 06.03.2014 14:09, julianfoad_at_apache.org wrote:
> Author: julianfoad
> Date: Thu Mar 6 13:09:26 2014
> New Revision: 1574868
>
> URL: http://svn.apache.org/r1574868
> Log:
> Don't let invalid mergeinfo stop 'svnadmin dump' from producing a dump.
> Just continue.

[...]

> + /* An error in verifying the mergeinfo must not prevent dumping
> + the data. Ignore any such error. */
> + svn_error_clear(verify_mergeinfo_revisions(
> + eb->found_old_mergeinfo,
> + mergeinfo_str->data, eb->oldest_dumped_rev,
> + eb->notify_func, eb->notify_baton,
> + pool));
> }
> }

Shouldn't we at least emit a warning here?

-- Brane

-- 
Branko Čibej | Director of Subversion
WANdisco // Non-Stop Data
e. brane_at_wandisco.com
Received on 2014-03-08 14:15:24 CET

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.