You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2014/04/11 17:07:23 UTC

svn commit: r1586676 - /subversion/branches/1.8.x/STATUS

Author: julianfoad
Date: Fri Apr 11 15:07:22 2014
New Revision: 1586676

URL: http://svn.apache.org/r1586676
Log:
* STATUS: Vote for r1580832, the svnrdump load segfault 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=1586676&r1=1586675&r2=1586676&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Fri Apr 11 15:07:22 2014
@@ -270,16 +270,6 @@ Candidate changes:
    Votes:
      +1: rhuijben
 
- * r1580832
-   svnrdump: resolve segfault on 'svnrdump load' deleting an 'svn:*' property
-   Issue #4490.
-   Justification:
-     Simple fix for a segfault
-   Votes:
-     +1: rhuijben, ivan
-     +0: julianfoad (Looks like an obvious fix. I would like to have a
-                     regression test; I'm working on one.)
-
  * r1573106, r1573209
    A more stringent test for whether the compiler accepts a given option.
    Justification:
@@ -386,6 +376,19 @@ Veto-blocked changes:
 Approved changes:
 =================
 
+ * r1580832
+   svnrdump: resolve segfault on 'svnrdump load' deleting an 'svn:*' property
+   Issue #4490.
+   Justification:
+     Simple fix for a segfault
+   Notes:
+     r1586668 on trunk is a regression test for this fix. Normally we'd
+     backport a regression test along with a fix but others have already
+     voted for the fix without it, so, as it's simple, I'm not bothering to
+     do so this time.
+   Votes:
+     +1: rhuijben, ivan, julianfoad
+
  * r1570642, r1585686, r1586352
    Fix memcached support.
    Justification: