Author: kotkov
Date: Thu Aug 21 14:21:34 2014
New Revision: 1619408

URL: http://svn.apache.org/r1619408
Log:
Following up on r1619068, do not mention the removed SVN_ERR_RECURSIVE_LOCK
error in comments.  We no longer used checked mutexes, so this part of the
information is now outdated, and we just have to tell the reader about the
possible deadlock and about how we avoid it.

No functional change.

* subversion/tests/cmdline/svnadmin_tests.py
  (freeze_freeze, freeze_same_uuid): Update comments.

Modified:
    subversion/trunk/subversion/tests/cmdline/svnadmin_tests.py

Modified: subversion/trunk/subversion/tests/cmdline/svnadmin_tests.py
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/tests/cmdline/svnadmin_tests.py?rev=1619408&r1=1619407&r2=1619408&view=diff
==============================================================================
--- subversion/trunk/subversion/tests/cmdline/svnadmin_tests.py (original)
+++ subversion/trunk/subversion/tests/cmdline/svnadmin_tests.py Thu Aug 21 
14:21:34 2014
@@ -2591,10 +2591,9 @@ def freeze_freeze(sbox):
     # FSFS repositories created with --compatible-version=1.8 and less
     # erroneously share the filesystem data (locks, shared transaction
     # data, ...) between hotcopy source and destination.  This is fixed
-    # for new FS formats, but in order to avoid SVN_ERR_RECURSIVE_LOCK
-    # for old formats, we have to manually assign a new UUID for the
-    # hotcopy destination.  As of trunk@1618024, the same applies to
-    # FSX repositories.
+    # for new FS formats, but in order to avoid a deadlock for old formats,
+    # we have to manually assign a new UUID for the hotcopy destination.
+    # As of trunk@1618024, the same applies to FSX repositories.
     svntest.actions.run_and_verify_svnadmin(None, [], None,
                                             'setuuid', second_repo_dir)
 
@@ -2906,10 +2905,9 @@ def freeze_same_uuid(sbox):
   first_repo_dir, _ = sbox.add_repo_path('first')
   second_repo_dir, _ = sbox.add_repo_path('second')
 
-  # Test that 'svnadmin freeze A (svnadmin freeze B)' does not deadlock or
-  # error out with SVN_ERR_RECURSIVE_LOCK for new FSFS formats, even if 'A'
-  # and 'B' share the same UUID.  Create two repositories by loading the
-  # same dump file, ...
+  # Test that 'svnadmin freeze A (svnadmin freeze B)' does not deadlock for
+  # new FSFS formats, even if 'A' and 'B' share the same UUID.  Create two
+  # repositories by loading the same dump file, ...
   svntest.main.create_repos(first_repo_dir)
   svntest.main.create_repos(second_repo_dir)
 


Reply via email to