James McCoy <jamessan@debian.org>


Found Patch Suggested
r1700740, r1696695, r1664997 r1714790, r1709389, r1700951, r1668340, r1596882, r1559009 r1567985

r1714790 | philip | 2015-11-17 08:20:09 -0600 (Tue, 17 Nov 2015)

Adapt Ruby tests to work with test-unit gem.

Ruby 2.2 bundles the test-unit gem instead of using the older "test-unit
API provided by minitest".  This also fixes testing when the test-unit
gem happens to be installed along with older Ruby versions.

* Makefile.in:
  (check-swig-rb): Detect whether test-unit supports the --collector
   argument and, if so, use the dir collector.

* subversion/bindings/swig/ruby/test/my-assertions.rb:
  (_my_assert_block): Remove unnecessary assert_block.

* subversion/bindings/swig/ruby/test/test_wc.rb:
  (test_context_create): Remove useless assertion on result of
   Svn::Wc::Create, especially since it varies based on Ruby version.

Patch by: James McCoy <jamessan{_AT_}debian.org>


r1709389 | rhuijben | 2015-10-19 06:05:57 -0500 (Mon, 19 Oct 2015)

Apply translation fixes supplied by the debian maintainer.

Resolves one mismatch in all languages where we changed a bit
of witespace and adds one french translation item.

* subversion/po/es.po
* subversion/po/fr.po
* subversion/po/it.po
* subversion/po/ja.po
* subversion/po/ko.po
* subversion/po/nb.po
* subversion/po/pl.po
* subversion/po/pt_BR.po
* subversion/po/sv.po
* subversion/po/zh_CN.po
* subversion/po/zh_TW.po
  Tweak commit notifications.

Patch by: James McCoy <jamessan{_AT_}debian.org>
Via https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=802156


r1700951 | rhuijben | 2015-09-03 03:29:26 -0500 (Thu, 03 Sep 2015)

Following up on r1700740, add missing include.

Patch by: James McCoy <jamessan{_AT_}debian.org>

* subversion/libsvn_auth_kwallet/kwallet.cpp
  (includes): Add svn_hash.h.


r1700740 | rhuijben | 2015-09-02 04:15:02 -0500 (Wed, 02 Sep 2015)

In the kwallet code: Initialize the wallet cache once, instead
of every time we get or save a password. Cache the wallet in the
parameter hash and its pool instead of the single invocation pool.

This patch is in response to an issue
Found by: James McCoy <jamessan{_AT_}debian.org>

* subversion/libsvn_auth_kwallet/kwallet.cpp
  (kwallet_terminate): Add forward definition.
  (get_wallet): Register pool cleanup handler once, and only when we set
    the wallet. Register cleanup on the hash pool.
  (kwallet_terminate): Completely cleanup.

  (kwallet_password_get,
   kwallet_password_set): Don't add another cleanup hook on every invocation.


r1696695 | rhuijben | 2015-08-19 20:10:22 -0500 (Wed, 19 Aug 2015)

Resolve a circular library reference issue in libsvn_fs_x, by using the passed
pointer to a function instead of the function by its name.

Found by: danielsh
          James McCoy <jamessan@debian.org>

* subversion/libsvn_fs_x/fs_id.c
  (svn_fs_x__id_context_t): Add function pointer.
  (fs_cleanup): Store open pointer to allow reopening.
  (get_fs): Use function pointer.


r1668340 | breser | 2015-03-21 23:17:04 -0500 (Sat, 21 Mar 2015)

Add missing POD directives to resolve pod2man warnings

* subversion/bindings/swig/perl/native/Core.pm
  (svn_log_entry_t): Add missing "=over 4" and "=back" directives

Patch by: James McCoy <jamessan{_AT_}debian.org>


r1664997 | rhuijben | 2015-03-08 09:29:26 -0500 (Sun, 08 Mar 2015)

Fix test failures when running from a directory whose name contains characters
which must be escaped in a URL. This is a regression caused by the new stricter
dump file parsing.

Subversion has its own rules about which characters needs escaping and as
we expect them to exactly match in test results we can't just use urllib
to perform the path escaping for us.

Found by: James McCoy <jamessan{_AT_}debian.org>
(The original patch broke the test framework on Windows and I decided to add a
 helper function instead of duplicating the same code in a few tests).

* subversion/tests/cmdline/svnsync_authz_tests.py
  (imports): Remove unneeded import.
  (copy_delete_unreadable_child): Update usage.

* subversion/tests/cmdline/svnsync_tests.py
  (imports): Remove unneeded import.
  (setup_and_sync): Update usage.

* subversion/tests/cmdline/svntest/main.py
  (file_scheme_prefix): Tweak Windows value.
  (_create_parser, execute_tests): Use svntest.wc.svn_uri_quote

* subversion/tests/cmdline/svntest/sandbox.py
  (imports): Remove unneeded import.
  (_set_name,
   add_repo_path): Use svntest.wc.svn_uri_quote
  (file_protocol_url): New function.


r1596882 | philip | 2014-05-22 09:15:45 -0500 (Thu, 22 May 2014)

Allow Swig to use Ruby 2.0 and 2.1.

Patch by: James McCoy <jamessan{_AT_}debian.org>

* build/ac-macros/swig.m4
  (SVN_FIND_SWIG): Adjust include path for Ruby 2.

* configure.ac: Allow Ruby 2.0 and 2.1.


r1567985 | stefan2 | 2014-02-13 11:48:59 -0600 (Thu, 13 Feb 2014)

On 32 bit systems, integer underflows can cause the membuffer usage counters
to grow beyond 32 bit limits.  That makes all entries appear to be "small"
and being kept instead of being potentially evicted.

* subversion/libsvn_subr/cache-membuffer.c
  (svn_membuffer_t): Update comment.
  (membuffer_cache_set_internal): Adding a cast to prevent off-by-4GB in the
                                  cache->DATA_USED tracker in 32 bits systems.

Found by: vitalif{_AT_}yourcmc.ru
Suggested by: James McCoy <jamessan{_AT_}debian.org>


r1559009 | breser | 2014-01-16 22:54:43 -0600 (Thu, 16 Jan 2014)

swig-pl: Fix when configure is passed --enable-sqlite-compatibility-version

* subversion/bindings/swig/perl/native/Makefile.PL.in
  ($cppflags, $cflags): Set from autoconf with here documents to avoid quoting
    issues.

Patch by: James McCoy <jamessan{_AT_}debian.org>