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

Re: svnadmin verify + rep-cache.db

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Wed, 13 Jul 2011 17:11:34 +0300

Does implementing this require API changes? (extending the fs-loader.h
vtables)

There is currently no svn_fs_verify() API.

Stefan Sperling wrote on Wed, Jul 13, 2011 at 15:59:36 +0200:
> Hi all,
>
> while repairing some bad repository breakage we noticed that
> svnadmin verify does not check whether offsets mentioned in
> rep-cache.db point to valid reps.
>
> In some cases I had to append repaired reps after the changed-paths
> section in revision files and leave broken reps in place to avoid
> adjusting offsets everywhere.
>
> So now 'svnadmin verify' runs fine. But when the rep-cache is
> referred to during commit we can still run into reps that are broken.
>
> Obviously we'll either have to fix up the rep-cache as well,
> or simply delete it. But the point is that 'svnadmin verify'
> does not complain.
>
> Would it be feasable to add this as a separate pass of 'svnadmin verify'?
> Should I file an issue about this?
>
> For instance, good output could look like this:
>
> $ svnadmin verify repos
> * Verified revision 1.
> ...
> * Verified revision N.
> * Verified rep-cache.
>
> $
>
> And errors like this:
>
> $ svnadmin verify repos
> * Verified revision 1.
> ...
> * Verified revision N.
> svnamdin: rep-cache entry '8fe47fa166154fc6e7f2e78366591c470ba75cf7' is invalid
> svnadmin: Malformed representation header
>
> $
Received on 2011-07-13 16:12:20 CEST

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