Author: philip
Date: Fri Feb 13 16:39:56 2015
New Revision: 1659597
URL: http://svn.apache.org/r1659597
Log:
* STATUS: Veto svnsync r0 mergeinfo fix.
Modified:
subversion/branches/1.8.x/STATUS
Modified: subversion/branches/1.8.x/STATUS
URL:
http://svn.apache.org/viewvc/subversion/branches/1.8.x/STATUS?rev=1659597&r1=1659596&r2=1659597&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Fri Feb 13 16:39:56 2015
@@ -112,22 +112,6 @@ Candidate changes:
Votes:
+1: philip, rhuijben
- * r1646230, r1646250, r1646253
- Make svnsync strip any r0 references from mergeinfo -- issue #4476.
- Justification:
- Without this is was not possible to svnsync a repository that contained
- this kind of mergeinfo that is now considered invalid but previously
- could be committed.
- Notes:
- r1646250 contains the fix itself, and a test for it.
- r1646253 removes some stray debug statements.
- r1646230 is required for the test to work.
- r1643074 (nominated separately) is required for the test to work.
- I would recommend not backporting r1646250 without r1643074.
- This is also nominated for 1.7.x.
- Votes:
- +1: julianfoad
-
* r1646785, r1646786, r1646797
Fix a pool handling bug in config variable expansion.
Justification:
@@ -231,6 +215,23 @@ Veto-blocked changes:
+1: julianfoad, rhuijben
-1: philip (uses apr_hash_key this without the neccessary backport
support)
+ * r1646230, r1646250, r1646253
+ Make svnsync strip any r0 references from mergeinfo -- issue #4476.
+ Justification:
+ Without this is was not possible to svnsync a repository that contained
+ this kind of mergeinfo that is now considered invalid but previously
+ could be committed.
+ Notes:
+ r1646250 contains the fix itself, and a test for it.
+ r1646253 removes some stray debug statements.
+ r1646230 is required for the test to work.
+ r1643074 (nominated separately) is required for the test to work.
+ I would recommend not backporting r1646250 without r1643074.
+ This is also nominated for 1.7.x.
+ Votes:
+ +1: julianfoad
+ -1: philip (deletes mergeinfo that doesn't contain r0)
+
Approved changes:
=================
