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