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

Re: svn commit: r1367683 - /subversion/trunk/subversion/libsvn_fs_fs/fs_fs.c

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Mon, 13 Aug 2012 08:39:38 +0100

Daniel Shahaf <d.s_at_daniel.shahaf.name> writes:

> philip_at_apache.org wrote on Tue, Jul 31, 2012 at 17:14:58 -0000:
>> Author: philip
>> Date: Tue Jul 31 17:14:58 2012
>> New Revision: 1367683
>>
>> URL: http://svn.apache.org/viewvc?rev=1367683&view=rev
>> Log:
>> Fix issue 4213, "svnadmin recover" fails on old format repositories.
>>
>> * subversion/libsvn_fs_fs/fs_fs.c
>> (open_pack_or_rev_file): Return SVN_ERR_FS_NO_SUCH_REVISION for
>> old format repositories.
>
>
> Worth a regression test?
>
> Index: subversion/tests/cmdline/svnadmin_tests.py
> ===================================================================
> --- subversion/tests/cmdline/svnadmin_tests.py (revision 1372130)
> +++ subversion/tests/cmdline/svnadmin_tests.py (working copy)
> @@ -1821,6 +1821,11 @@ def mergeinfo_race(sbox):
> raise svntest.Failure("one or both commits failed")
>
>
> +@Issue(4213)
> +def recover_old(sbox):
> + "recover --pre-1.4-compatible"
> + sbox.build(create_wc=False)
> + svntest.main.run_svnadmin("recover", sbox.repo_dir)
>
>
> ########################################################################
> @@ -1858,6 +1863,7 @@ test_list = [ None,
> hotcopy_incremental_packed,
> locking,
> mergeinfo_race,
> + recover_old,
> ]
>
> if __name__ == '__main__':
>

Looks good.

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download
Received on 2012-08-13 09:40:15 CEST

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.