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/08/12 22:20:28 UTC

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

Author: rhuijben
Date: Mon Aug 12 20:20:28 2013
New Revision: 1513243

URL: http://svn.apache.org/r1513243
Log:
* STATUS: Cast some votes. Approve diff error reporting. Update nomination text

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=1513243&r1=1513242&r2=1513243&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Mon Aug 12 20:20:28 2013
@@ -193,14 +193,14 @@ Candidate changes:
    Justification:
      It's an assertion failure.
    Votes:
-     +1: danielsh
+     +1: danielsh, rhuijben
 
  * r1508438
    Try harder to pass an existing path to sleep for timestamps
    Justification:
      Performance regression from 1.6, reported by user.
    Votes:
-     +1: philip
+     +1: philip, rhuijben
 
  * r1512300, r1512301
    Clean up unpacked revprops in 'svnadmin hotcopy --incremental'
@@ -213,15 +213,6 @@ Candidate changes:
    Votes:
      +1: stefan2
 
- * r1513119, r1513122
-   Correctly error out for large files in svn_diff_file_output_merge2() on
-   32-bit platforms.
-   Justification:
-     Silently converting file size to apr_size_t is definitely wrong. Got
-     related crashes in TortoiseSVN code.
-   Votes:
-     +1: ivan, philip
-
  * r1513156
    Fix crash when updating in libsvn_wc/update_editor.c, window_handler().
    Justification:
@@ -263,10 +254,21 @@ Approved changes:
 =================
 
  * r1507382, r1509186, r1509196
-   Allow passing NULL for the config dir to use the default config dir path.
-   r1509186 is to update the docstring accordingly.
+   Allow passing NULL for the config dir to svn_config_walk_auth_data to use
+   the default config dir path, like other functions handle this.
    Justification: all other APIs that take a config dir param allow NULL as
      meaning the default config path. Clients should not try to figure out
      that path themselves.
+   Notes:
+      r1509186 and r1509196 update the docstring accordingly.
+   Votes:
+     +1: steveking, philip, ivan, rhuijben
+
+ * r1513119, r1513122
+   Correctly error out for large files in svn_diff_file_output_merge2() on
+   32-bit platforms.
+   Justification:
+     Silently converting file size to apr_size_t is definitely wrong. Got
+     related crashes in TortoiseSVN code.
    Votes:
-     +1: steveking, philip, ivan
+     +1: ivan, philip, rhuijben