Arwin Arni <>

r1124283, r1102690, r1099836, r1099758, r1095662, r1086537, r1081390, r1069821, r1069791, r1064093, r1060836, r1055463, r1054090, r1053833, r1049819, r1032610

r1124283 | pburba | 2011-05-18 09:33:11 -0500 (Wed, 18 May 2011)

Add a test for issue #3867 'reintegrate merges create mergeinfo for
non-existent paths'

* subversion/tests/cmdline/
  (reintegrate_creates_bogus_mergeinfo) : New XFail test for Issue #3867.

Patch by: Arwin Arni <arwin{_AT_}>
          (tweaked by me)

r1102690 | kameshj | 2011-05-13 07:22:15 -0500 (Fri, 13 May 2011)

Fix for issue3870 "File descriptor leaks during svnsync".

Before this fix, all of destination delta editor's interfaces
are called with *LONG* living pool(dst_rev_pool living for one full revision).

This makes it a memory bloat and bloat of other OS resources like
file descriptors to live as long the dst_rev_pool.

* subversion/libsvn_ra_serf/replay.c
  (replay_context_t.file_pool): New pool of file scope.
  (start_replay): clear the file_pool.
                  Use file_pool for dest editor's file operations.
  (end_replay): Use file_pool for dest editor's file operations.
  (svn_ra_serf__replay, svn_ra_serf__replay_range): 
   Create a new pool 'replay_ctx->file_pool'.

* subversion/tests/cmdline/
  (fd_leak_sync_from_serf_to_local): Remove XFail marker.
Patch by: kameshj
          Arwin Arni <arwin{_AT_}>

r1099836 | philip | 2011-05-05 10:07:22 -0500 (Thu, 05 May 2011)

Add XFAIL test for issue 3870, sync from ra_serf to ra_local.

* subversion/tests/cmdline/
  (setup_and_sync): Modified to accept two bools 'is_src_ra_local' and 
                    'is_dest_ra_local' to sync from or to local URLs.
  (run_test):       Percolate is_src_ra_local and is_dest_ra_local.
                    Fix call of setup_and_sync.
  (fd_leak_sync_from_serf_to_local): New.

* subversion/tests/cmdline/svnsync_tests_data/largemods.dump: New.

Patch by: Arwin Arni <arwin{_AT_}>

r1099758 | rhuijben | 2011-05-05 06:38:39 -0500 (Thu, 05 May 2011)

* subversion/libsvn_ra/util.c
  (svn_ra__release_operational_lock): Handle a previously unhandled error.

Patch by: Arwin Arni <arwin{_AT_}>
          (tweaked by me)

r1095662 | kameshj | 2011-04-21 06:42:32 -0500 (Thu, 21 Apr 2011)

* subversion/tests/cmdline/
  Add a test-case to showcase an out-of-date slave related issue #3680.

Patch by: Arwin Arni <arwin{_AT_}>

r1086537 | danielsh | 2011-03-29 05:20:43 -0500 (Tue, 29 Mar 2011)

    Bump serf version to 0.7.1.

Patch by: Arwin Arni <arwin{_AT_}>

r1081390 | kameshj | 2011-03-14 09:24:58 -0500 (Mon, 14 Mar 2011)

Adds an XFail test to catch regression created by r1075802

* subversion/tests/cmdline/
  (dry_run_merge_conflicting_binary): New XFail testcase.
  (test_list): Add dry_run_merge_conflicting_binary.

Patch by: Arwin Arni <arwin{_AT_}>

r1069821 | kameshj | 2011-02-11 08:44:13 -0600 (Fri, 11 Feb 2011)

Follow-up to r1069791. Fix deprecated calls of svn_config_read.

* subversion/libsvn_subr/config.c
  (read_all, svn_config_merge):
* subversion/tests/libsvn_subr/cache-test.c
  (test_memcache_basic, test_memcache_longkey):
* subversion/tests/libsvn_subr/config-test.c
  (test_text_retrieval, test_boolean_retrieval, test_has_section):
* subversion/svnserve/serve.c
* subversion/libsvn_fs_fs/fs_fs.c
   Fixed callers.

Patch by: Arwin Arni <arwin{_AT_}>

r1069791 | kameshj | 2011-02-11 07:18:15 -0600 (Fri, 11 Feb 2011)

Fix Issue #3781 (Case sensitive authz).

* subversion/tests/cmdline/
  (case_sensitive_authz): Removed XFail decorator.

* subversion/include/svn_config.h
  (svn_config_create) : Documented new parameter.
  (svn_config_read)   : Deprecated.
  (svn_config_read2)  : New function prototype, updated documentation.

* subversion/libsvn_subr/config_impl.h
  (svn_config_t): Added a new boolean called section_names_case_sensitive.

* subversion/libsvn_subr/deprecated.c
  (svn_config_read): New function that calls svn_config_read2
                     with section_names_case_sensitive as FALSE.

* subversion/libsvn_subr/config.c
  (svn_config_create): Added a parameter section_names_case_sensitive
                       which is used to initialize the config.
   svn_config_set)   : Added logic to make section names case sensitive.

  (svn_config_read)  : Deprecated.

  (svn_config_read2) : New function that deprecates svn_config_read.
                       Accepts a boolean section_names_case_sensitive,
                       and populates the section names case sensitively 
                       if TRUE.

* subversion/libsvn_repos/authz.c
  (svn_repos_authz_read) : Fixed the caller with 
                           section_names_case_sensitive as TRUE.
* subversion/tests/cmdline/atomic-ra-revprop-change.c
  (construct_config) : Fixed call of svn_config_create.
Patch by: Arwin Arni <arwin{_AT_}>

r1064093 | kameshj | 2011-01-27 05:41:43 -0600 (Thu, 27 Jan 2011)

Fix for Issue #3781 repo prefix rules in authz section is checked case
sensitively for write operations

* subversion/libsvn_repos/authz.c
  (authz_get_any_access_parser_cb): Use strncasecmp() instead of strncmp()

Patch by: me
          Arwin Arni <arwin{_AT_}>

Reverted in r1064729

r1060836 | cmpilato | 2011-01-19 09:29:56 -0600 (Wed, 19 Jan 2011)

Changed some function names to better describe their behavior.

* subversion/libsvn_repos/authz.c
  (authz_get_any_access): Renamed from authz_get_global_access(), and
    added an explanatory comment.  Callers updated.
  (authz_get_any_access_parser_cb): Renamed from
    authz_global_parse_section().  Callers updated.

Patch by: Arwin Arni <arwin{_AT_}>
          (Tweaked by me.)

r1055463 | philip | 2011-01-05 08:15:39 -0600 (Wed, 05 Jan 2011)

Removed a redundant member (skip_descendants) from the dir baton, the
skip_this flag is sufficient.

* subversion/libsvn_wc/update_editor.c
  (struct dir_baton): Removed 'skip_descendants'.
  (open_root, delete_entry, open_file, add_directory,
   open_directory): Removed unreachable code.

Patch by: Arwin Arni <>

r1054090 | kameshj | 2010-12-31 05:07:08 -0600 (Fri, 31 Dec 2010)

Adds an XFail test of wc-upgrade(1.6-wc+scheduled_change to 1.7-wc).

* subversion/tests/cmdline/upgrade_tests_data/upgrade_with_scheduled_change.tar.bz2
  New file.

* subversion/tests/cmdline/
  (upgrade_with_scheduled_change): New XFail testcase.

Patch by: Arwin Arni <arwin{_AT_}>

r1053833 | kameshj | 2010-12-30 03:10:08 -0600 (Thu, 30 Dec 2010)

Removed unused function parameters.

* subversion/libsvn_wc/update_editor.c
  (accumulate_last_change): Removed 'svn_wc__db_t *db'
   and 'const char *local_abspath' from function signature.
  (close_directory, close_file, svn_wc_add_repos_file4): Adjust for above

Patch by: Arwin Arni <arwin{_AT_}>

r1049819 | kameshj | 2010-12-16 02:45:46 -0600 (Thu, 16 Dec 2010)

Removed a redundant member (added_with_history) from the file baton
and removed *relevent* unreachable code.  A follow-up to r998193.

* subversion/libsvn_wc/update_editor.c
  (struct file_baton): Removed 'added_with_history'.
  (merge_file, close_file): Removed unreachable code.

Patch by: Arwin Arni <arwin{_AT_}>

r1032610 | kameshj | 2010-11-08 09:46:00 -0600 (Mon, 08 Nov 2010)

Improved the documentation of deprecated functions by providing links
to the deprecating functions.

* subversion/include/svn_path.h
  (svn_path_internal_style, svn_path_local_style,
   svn_path_join, svn_path_join_many,
   svn_path_basename, svn_path_dirname,
   svn_path_split, svn_path_canonicalize,
   svn_path_is_canonical, svn_path_get_longest_ancestor,
   svn_path_get_absolute, svn_path_split,
   svn_path_condense_targets, svn_path_is_child,
   svn_path_is_ancestor, svn_path_url_add_component): Provide reference to 
   deprecating functions.

Patch by: Arwin Arni <arwin{_AT_}>