You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by st...@apache.org on 2013/10/09 06:37:07 UTC
svn commit: r1530490 - /subversion/branches/1.8.x/STATUS
Author: stefan2
Date: Wed Oct 9 04:37:06 2013
New Revision: 1530490
URL: http://svn.apache.org/r1530490
Log:
Review spree.
* 1.8.x/STATUS:
- vote for r1518184, r1524869
- vote for & approve r1516806, r1519615 group, r1519955,
r1520529 group, r1522892, r1523666
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=1530490&r1=1530489&r2=1530490&view=diff
==============================================================================
--- subversion/branches/1.8.x/STATUS (original)
+++ subversion/branches/1.8.x/STATUS Wed Oct 9 04:37:06 2013
@@ -81,20 +81,6 @@ Candidate changes:
detection of BDB (via db.h) and even while debugging a
maintainer build of Subversion a release BDB works fine)
- * r1516806
- Do not look at the hostname part of an URI when checking that
- the schema data is canonical.
- Justification:
- As reported on dev@, "svn co file://./" and similar commands abort
- instead of returning an error that the URL is invalid. This was
- caused by svn_uri_is_canonical treating the hostname "." as a path
- segment. After this change, svn_uri_is_canonicall correctly
- determines that the URL is (syntactically) canonical, and the
- commands later faile with a normal error in the RA layer, which
- validates the semantics of the hostname.
- Votes:
- +1: brane, rhuijben
-
* ^/subversion/branches/1.8.x-serf-1.3+-windows
Allow compiling against serf 1.3 and later on Windows
Justification:
@@ -112,8 +98,69 @@ Candidate changes:
Justification:
Improves correctness. Resolves user reported problem.
Votes:
+ +1: rhuijben, stefan2
+
+ * r1520065, r1520745
+ Resolve a user triggerable assertion when during an update a file external
+ is being replaced by a server excluded (aka absent) node.
+ Justification:
+ Quite a few reports of this assertion on the users@s.a.o list since 1.7.0.
+ Votes:
+ +1: rhuijben
+
+ * r1524145
+ Make the 'm' conflict prompt option run external merge tool if configured.
+ Justification:
+ Usability fix. The current behaviour ('m' -> internal merge tool,
+ 'l' -> external merge tool) confuses users who try 'm' to run an
+ external merge tool. See issue #4426.
+ Votes:
+ +1: stsp
+
+ * r1526439
+ Improve http status processing during commit using ra_serf to improve
+ diagnostics.
+ Justification:
+ With this patch the problem described as issue #4431, would
+ probably have been diagnosed months ago.
+ Votes:
+1: rhuijben
+ * r1527103, r1527105
+ Fix OOM with concurrent requests right after threaded server start.
+ Justification:
+ This was a common condition in my load tests. Affects threaded
+ servers only. Apache will just respawn a new worker process.
+ Votes:
+ +1: stefan2
+
+ * r1524869
+ JavaHL: Canonicalize path for streamFileContent method.
+ Justification:
+ JavaHL is designed around us doing the canoicalization.
+ Votes:
+ +1: breser, stefan2
+
+Veto-blocked changes:
+=====================
+
+Approved changes:
+=================
+
+ * r1516806
+ Do not look at the hostname part of an URI when checking that
+ the schema data is canonical.
+ Justification:
+ As reported on dev@, "svn co file://./" and similar commands abort
+ instead of returning an error that the URL is invalid. This was
+ caused by svn_uri_is_canonical treating the hostname "." as a path
+ segment. After this change, svn_uri_is_canonicall correctly
+ determines that the URL is (syntactically) canonical, and the
+ commands later faile with a normal error in the RA layer, which
+ validates the semantics of the hostname.
+ Votes:
+ +1: brane, rhuijben, stefan2
+
* r1519615, r1519617, r1519733
Fix a memory problem in the 3rd party FS module loader.
Changes[dev]:
@@ -121,14 +168,14 @@ Candidate changes:
Justification:
Not enabled in default build but a SEGV likely when triggered.
Votes:
- +1: philip, danielsh
+ +1: philip, danielsh, stefan2
* r1519955
Fix a segfault with 'svn mergeinfo'.
Justification:
Missing NULL pointer checks are bad, user complained.
Votes:
- +1: stsp, rhuijben
+ +1: stsp, rhuijben, stefan2
* r1520529, r1520532, r1520539
Properly re-canonicalize urls read from xml based entries files.
@@ -137,15 +184,7 @@ Candidate changes:
we fixed the newer entries file parsing we forgot the xml based
entries. Fixes a client assert.
Votes:
- +1: rhuijben, philip
-
- * r1520065, r1520745
- Resolve a user triggerable assertion when during an update a file external
- is being replaced by a server excluded (aka absent) node.
- Justification:
- Quite a few reports of this assertion on the users@s.a.o list since 1.7.0.
- Votes:
- +1: rhuijben
+ +1: rhuijben, philip, stefan2
* r1522892
Fix issue #4425 and #3014 on Windows: svn.exe crashed when output of 'log'
@@ -156,7 +195,7 @@ Candidate changes:
Justification:
User reported crash.
Votes:
- +1: ivan, rhuijben
+ +1: ivan, rhuijben, stefan2
* r1523666
Fix path corruption in the calculation of reintegration merges.
@@ -170,43 +209,5 @@ Candidate changes:
and look for bad paths in 'Missing ranges' lines, e.g. '.../svnmucc.'
which should be '.../svnmucc.c'.
Votes:
- +1: rhuijben, stsp
-
- * r1524145
- Make the 'm' conflict prompt option run external merge tool if configured.
- Justification:
- Usability fix. The current behaviour ('m' -> internal merge tool,
- 'l' -> external merge tool) confuses users who try 'm' to run an
- external merge tool. See issue #4426.
- Votes:
- +1: stsp
+ +1: rhuijben, stsp, stefan2
- * r1526439
- Improve http status processing during commit using ra_serf to improve
- diagnostics.
- Justification:
- With this patch the problem described as issue #4431, would
- probably have been diagnosed months ago.
- Votes:
- +1: rhuijben
-
- * r1527103, r1527105
- Fix OOM with concurrent requests right after threaded server start.
- Justification:
- This was a common condition in my load tests. Affects threaded
- servers only. Apache will just respawn a new worker process.
- Votes:
- +1: stefan2
-
- * r1524869
- JavaHL: Canonicalize path for streamFileContent method.
- Justification:
- JavaHL is designed around us doing the canoicalization.
- Votes:
- +1: breser
-
-Veto-blocked changes:
-=====================
-
-Approved changes:
-=================