Re: [PATCH] Issue #1074: Need a svnadmin command to verify that the repository is not corrupted

Hey,

>Now, about that docstring for svn_fs_verify(). Seems a little
>lightweight for a module API docstring, but more than that -- it's
>kinda wrong. This function doesn't actually verify all the filesystem
>data. It only verifies the file contents, file and directory property
>lists, and directory entries lists.
>
>How about if we call the function svn_fs_verify_contents(), and
>compose a slightly more revealing docstring for it?
>
>

How about leaving it svn_fs_verify() and adding an argument like
verification level.

SVN_FS_V_EXHAUSTIVE
SVN_FS_V_FILE_CONTENT
SVN_FS_V_ANCESTRY
....
SVN_FS_V_WHY_BOTHER

Having verification levels to be implemented at FS implementors discretion.

gat

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 22 17:54:01 2003