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 2018/11/27 15:21:43 UTC
svn commit: r1847556 - in /subversion/branches/1.10.x: STATUS
subversion/mod_dav_svn/reports/list.c
Author: rhuijben
Date: Tue Nov 27 15:21:43 2018
New Revision: 1847556
URL: http://svn.apache.org/viewvc?rev=1847556&view=rev
Log:
Revert r1847555, after &*($%#
Modified:
subversion/branches/1.10.x/STATUS
subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c
Modified: subversion/branches/1.10.x/STATUS
URL: http://svn.apache.org/viewvc/subversion/branches/1.10.x/STATUS?rev=1847556&r1=1847555&r2=1847556&view=diff
==============================================================================
--- subversion/branches/1.10.x/STATUS (original)
+++ subversion/branches/1.10.x/STATUS Tue Nov 27 15:21:43 2018
@@ -22,12 +22,6 @@ Candidate changes:
Votes:
+1: brane
-Veto-blocked changes:
-=====================
-
-Approved changes:
-=================
-
* r1846299
Add resolver support for unversioned directories during update/switch.
Justification:
@@ -36,11 +30,17 @@ Approved changes:
Branch:
^/subversion/branches/1.10.x-r1846299
Votes:
- +1: stsp, brane, rhuijben
+ +1: stsp, brane
* r1847181, r1847182, r1847188, r1847264
Fix issue SVN-4792: Foreign repo copy of file adding mergeinfo.
Justification:
We don't want bogus mergeinfo.
Votes:
- +1: julianfoad, brane, rhuijben
\ No newline at end of file
+ +1: julianfoad, brane
+
+Veto-blocked changes:
+=====================
+
+Approved changes:
+=================
Modified: subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c
URL: http://svn.apache.org/viewvc/subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c?rev=1847556&r1=1847555&r2=1847556&view=diff
==============================================================================
--- subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c (original)
+++ subversion/branches/1.10.x/subversion/mod_dav_svn/reports/list.c Tue Nov 27 15:21:43 2018
@@ -201,7 +201,7 @@ dav_svn__list_report(const dav_resource
dav_svn__authz_read_baton arb;
const dav_svn_repos *repos = resource->info->repos;
int ns;
- const char *full_path = NULL;
+ const char *full_path;
svn_boolean_t path_info_only;
svn_fs_root_t *root;
svn_depth_t depth = svn_depth_unknown;
@@ -280,12 +280,6 @@ dav_svn__list_report(const dav_resource
/* else unknown element; skip it */
}
- if (! full_path)
- {
- return dav_svn__new_error_svn(resource->pool, HTTP_BAD_REQUEST, 0, 0,
- "Request was missing the path argument");
- }
-
/* Build authz read baton */
arb.r = resource->info->r;
arb.repos = resource->info->repos;