You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by rh...@apache.org on 2013/10/26 23:51:33 UTC

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

Author: rhuijben
Date: Sat Oct 26 21:51:33 2013
New Revision: 1536039

URL: http://svn.apache.org/r1536039
Log:
* STATUS: Approve r1501656 group; updating my original comment.

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=1536039&r1=1536038&r2=1536039&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Sat Oct 26 21:51:33 2013
@@ -58,19 +58,6 @@ Candidate changes:
      +1: stsp, rhuijben
      -0: ivan (not regression from 1.7.x, destructive change in libsvn_wc core)
 
- * r1501656, r1501702
-   Fix the ability to build Debug Build with BDB FS on Windows.
-   Justification:
-     Our build system should work properly.
-   Votes:
-     +1: breser, stefan2
-     -0: rhuijben (Not a regression and I don't think we should encourage
-                   building with just a debug BDB (even for release mode)
-                   by backporting this change. That would require using the
-                   non-distributable debug CRT. Trunk has a cleaner
-                   detection of BDB (via db.h) and even while debugging a
-                   maintainer build of Subversion a release BDB works fine)
-
  * r1524145
    Make the 'm' conflict prompt option run external merge tool if configured.
    Justification:
@@ -157,3 +144,12 @@ Veto-blocked changes:
 
 Approved changes:
 =================
+
+ * r1501656, r1501702
+   Fix the ability to build Debug Build with BDB FS on Windows.
+   Justification:
+     Our build system should work properly.
+   Votes:
+     +1: breser, stefan2
+     +1: rhuijben (I prefer the trunk code that performs a real bdb install
+                   check, but it might help in some dev environments)