Author: danielsh
Date: Tue Jul 9 16:43:46 2013
New Revision: 1501374
URL: http://svn.apache.org/r1501374
Log:
Nominate r1501371.
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=1501374&r1=1501373&r2=1501374&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Tue Jul 9 16:43:46 2013
@@ -137,7 +137,7 @@ Veto-blocked changes:
consensus)
-1: (not vetoed, but depends on the r1489117 group which is)
- * r1501049
+ * r1501049, r1501371
ra_serf: Return Subversion error codes in svn_error_t.
Justification:
Other error codes are useless to API users.
@@ -152,7 +152,13 @@ Veto-blocked changes:
The reason for the -1 is the re-use of a specific error code
that is automatically unwrapped in some ra_serf code, to
avoid
- handling codes like APR_EOF as non fatal)
+ handling codes like APR_EOF as non fatal
+ [ without r1501371 ]
+ <danielsh>: Responded in detail in the r1501049 thread.
+ tldr: outermost svn_error_t->apr_err should be defined by
+ Subversion or by APR, not by serf; can't find evidence of
+ the "automatic unwrapping", and in any case happy to change
+ the error code to something else if that's the problem.)
Approved changes:
=================
