You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ph...@apache.org on 2011/08/24 12:19:40 UTC

svn commit: r1161040 - /subversion/branches/1.7.x/STATUS

Author: philip
Date: Wed Aug 24 10:19:39 2011
New Revision: 1161040

URL: http://svn.apache.org/viewvc?rev=1161040&view=rev
Log:
* STATUS: Vote/approve r1158617 et al.

Modified:
    subversion/branches/1.7.x/STATUS

Modified: subversion/branches/1.7.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.7.x/STATUS?rev=1161040&r1=1161039&r2=1161040&view=diff
==============================================================================
--- subversion/branches/1.7.x/STATUS (original)
+++ subversion/branches/1.7.x/STATUS Wed Aug 24 10:19:39 2011
@@ -69,29 +69,6 @@ Candidate changes:
    Votes:
      +1: rhuijben, cmpilato
 
-  * r1158617, r1158875, r1158886, r1158896, r1158919, r1158924, r1158963
-    Make 'svn revert' remove unversioned nodes resulting from reverting a copy.
-    Fixes issue #3101.
-    Justification:
-      Bert suggested to backport this to 1.7.
-      I myself am not really sure if we should consider this a bug fix
-      or a behaviour change. I think we should merge this before 1.7.0
-      if we decide to merge it.
-      It does fix some fairly annoying usability issues.
-      For instance: If a merge adds files, the merge result is reverted, and
-      the merge is repeated, the second merge does not produce the same result
-      as the first merge. It will skip the, now unversioned, files added by
-      the first merge. There are various other cases where the unversioned
-      files get in the way.
-    Notes:
-      All revisions depend on one another, but some amend or even undo changes
-      made in earlier revisions. Instead of reviewing the individual commits
-      I would suggest to run the following merge and review the resulting diff:
-        svn merge -c1158617,1158875,1158886,1158896,1158919,1158924,1158963 \
-          ^/subversion/trunk
-    Votes:
-      +1: stsp, rhuijben (but only if this gets into 1.7.0)
-
  * r1159400
    Document a neat trick to better handle patch rejects in 'svn help patch'.
    Justification:
@@ -147,3 +124,26 @@ Approved changes:
    Votes:
      +1: danielsh (without r1158187, so just the tests and not the fix)
      +1: rhuijben, cmpilato, philip
+
+  * r1158617, r1158875, r1158886, r1158896, r1158919, r1158924, r1158963
+    Make 'svn revert' remove unversioned nodes resulting from reverting a copy.
+    Fixes issue #3101.
+    Justification:
+      Bert suggested to backport this to 1.7.
+      I myself am not really sure if we should consider this a bug fix
+      or a behaviour change. I think we should merge this before 1.7.0
+      if we decide to merge it.
+      It does fix some fairly annoying usability issues.
+      For instance: If a merge adds files, the merge result is reverted, and
+      the merge is repeated, the second merge does not produce the same result
+      as the first merge. It will skip the, now unversioned, files added by
+      the first merge. There are various other cases where the unversioned
+      files get in the way.
+    Notes:
+      All revisions depend on one another, but some amend or even undo changes
+      made in earlier revisions. Instead of reviewing the individual commits
+      I would suggest to run the following merge and review the resulting diff:
+        svn merge -c1158617,1158875,1158886,1158896,1158919,1158924,1158963 \
+          ^/subversion/trunk
+    Votes:
+      +1: stsp, rhuijben (but only if this gets into 1.7.0), philip