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

Re: [PATCH] Implement svnadmin verify --force

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sun, 4 Nov 2012 02:32:25 +0200

> Index: subversion/libsvn_repos/dump.c
> ===================================================================
> --- subversion/libsvn_repos/dump.c (revision 1402414)
> +++ subversion/libsvn_repos/dump.c (working copy)
> @@ -1459,5 +1512,8 @@
> /* Per-backend verification. */
> svn_pool_destroy(iterpool);
>
> + if (found_corruption)
> + return svn_error_createf(SVN_ERR_REPOS_CORRUPTED, NULL,
> + _("Repository has corruptions."));

You don't know that the repository is corrupt (maybe the error was EPERM
on the db/ dir). "Repository '%s' failed to verify" would be better.
Received on 2012-11-04 01:33:03 CET

This is an archived mail posted to the Subversion Dev mailing list.