Roderich Schupp <>

Patch Suggested
r1161080, r1149572, r1103665, r935631, r933299 r1149240

r1161080 | philip | 2011-08-24 07:48:57 -0500 (Wed, 24 Aug 2011)

Make svnmucc honor option --config-dir for auth stuff.

* tools/client-side/svnmucc/svnmucc.c
  (create_ra_callbacks) Add parameter config_dir, pass it to
  (execute) Pass config_dir to create_ra_callbacks.

Patch by: Roderich Schupp <>

r1149572 | philip | 2011-07-22 06:33:14 -0500 (Fri, 22 Jul 2011)

Fix a server SEGV when AuthzForceUsernameCase is used without

* subversion/mod_authz_svn/mod_authz_svn.c
  (get_username_to_authorize): Allow NULL username.

Patch by: roderich.schupp{_AT_}

r1149240 | philip | 2011-07-21 10:53:44 -0500 (Thu, 21 Jul 2011)

Support APR 0.9.x.

* subversion/include/private/svn_dep_compat.h

* subversion/include/private/svn_string_private.h
  (svn__strtoff): New.

* subversion/libsvn_subr/svn_string.c
  (svn__strtoff): New.

* subversion/mod_dav_svn/util.c
  (request_body_to_string): Use svn__strtoff.

* subversion/tests/libsvn_wc/op-depth-test.c
  (): Include svn_dep_compat.h.

Suggested by: roderich.schupp{_AT_}

r1103665 | hwright | 2011-05-16 04:51:39 -0500 (Mon, 16 May 2011)

Fix a user-reported performance issue where we don't properly retrieve the
cached manifest offset value in FSFS.  See

Even though on trunk we are currently using apr_int64_t, danielsh tells me
that svn_revnum_t is a "more proper" type.

Patch by: Roderich Schupp <>

* subversion/libsvn_fs_fs/fs_fs.c
  (get_packed_offset): Use the proper type for the shard.

r935631 | cmpilato | 2010-04-19 10:46:35 -0500 (Mon, 19 Apr 2010)

Fix check in "merge --reintegrate" whether merge source and
working copy target belong to the same repository.

* subversion/libsvn_client/merge.c
  (merge_reintegrate_locked) Correctly compute the repos root URL for
    the merge source.  Add FIXME comment regarding possible
    unnecessary opening of an RA session at the repos root.

Patch by: Roderich Schupp <roderich.schupp{_AT_}>

r933299 | cmpilato | 2010-04-12 11:17:23 -0500 (Mon, 12 Apr 2010)

Fix another scenario related to issue #3242:

   - user has no access in repo root, only in /project
   - someone else renames some file below /project and commits
   - user tries to update his working copy of /project
     (that still has the old name)
   - update deletes the old name and then fails with
     "access to 'http://host/repo-root' forbidden"

* subversion/libsvn_client/update.c
  (struct ff_baton): Update comment.
  (file_fetcher): Open (or reparent, if already opened) auxiliary RA
    session at parent directory of to-be-fetched file instead of at
    the repository root.

* subversion/tests/cmdline/
  (authz_access_required_at_repo_root2): New test.
  (test_list): Add reference to new test.

Patch by: Roderich Schupp <roderich.schupp{_AT_}>
          (Tweaked by me.)