svn commit: r1036518 - /subversion/branches/1.6.x/STATUS

[rh...@apache.org]

Author: rhuijben
Date: Thu Nov 18 16:25:30 2010
New Revision: 1036518

URL: http://svn.apache.org/viewvc?rev=1036518&view=rev
Log:
* STATUS: Cast vote, approving r991972

Modified:
    subversion/branches/1.6.x/STATUS

Modified: subversion/branches/1.6.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.6.x/STATUS?rev=1036518&r1=1036517&r2=1036518&view=diff
==============================================================================
--- subversion/branches/1.6.x/STATUS (original)
+++ subversion/branches/1.6.x/STATUS Thu Nov 18 16:25:30 2010
@@ -106,25 +106,6 @@ Candidate changes:
    Votes:
      +1: danielsh
 
-  * r991972
-   Replace os.rename(src,dst) with shutil.move() while setting up authz files
-   in the test suite.
-   Notes:
-     On most linux variants, os.rename(src,dst) will tolerate an existing
-     dst, but Windows will not.  The change to use os.rename() was made in
-     the backport of r991534.
-   Justification:  
-     With r991534 backported, we can no longer run the dav tests on Windows
-     without this change.
-   Branch:
-     No branch is required, however r991972 was made in sandbox.py, which
-     doesn't exist on 1.6.x.  A file merge from sandbox.py to main.py on
-     the 1.6.x branch will cleanly apply the one line change, i.e.:     
-     svn merge ^/subversion/trunk/subversion/tests/cmdline/svntest/sandbox.py
-     subversion\tests\cmdline\svntest\main.py -c991972
-   Votes:
-     +1: pburba, stsp
-
  * r1036429
    Fix svnsync_tests 29 for BDB.
    Justification:
@@ -228,3 +209,22 @@ Veto-blocked changes:
 
 Approved changes:
 =================
+
+  * r991972
+   Replace os.rename(src,dst) with shutil.move() while setting up authz files
+   in the test suite.
+   Notes:
+     On most linux variants, os.rename(src,dst) will tolerate an existing
+     dst, but Windows will not.  The change to use os.rename() was made in
+     the backport of r991534.
+   Justification:  
+     With r991534 backported, we can no longer run the dav tests on Windows
+     without this change.
+   Branch:
+     No branch is required, however r991972 was made in sandbox.py, which
+     doesn't exist on 1.6.x.  A file merge from sandbox.py to main.py on
+     the 1.6.x branch will cleanly apply the one line change, i.e.:     
+     svn merge ^/subversion/trunk/subversion/tests/cmdline/svntest/sandbox.py
+     subversion\tests\cmdline\svntest\main.py -c991972
+   Votes:
+     +1: pburba, stsp, rhuijben