Alexey Neyman <>

Found Patch Suggested
r1487987, r917640 r1511353, r1443821, r1380697, r1239553, r984565, r927222 r1487842, r1442645, r1442640

r1511353 | danielsh | 2013-08-07 10:11:04 -0500 (Wed, 07 Aug 2013)

svnmucc: Fix an assertion failure.

* subversion/svnmucc/svnmucc.c
  (main): Check for URLs not having a common ancestor and emit an error
    message instead of failing an assertion later.

Patch by: Alexey Neyman <>
(tweaked by me)

r1487987 | cmpilato | 2013-05-30 14:38:51 -0500 (Thu, 30 May 2013)

* site/publish/index.html,
* site/publish/news.html
  Fix timestamps of Hackathon news item(s).

Reported by: Alexey Neyman <stilor{_AT_}>

r1487842 | cmpilato | 2013-05-30 08:02:36 -0500 (Thu, 30 May 2013)

* site/publish/roadmap.html
  Add link to issue #1973 for the "log message templates" feature.

Suggested by: Alexey Neyman <stilor{_AT_}>

r1443821 | julianfoad | 2013-02-07 19:56:29 -0600 (Thu, 07 Feb 2013)

Make 'svn diff --old=FOO --new=BAR' default to WORKING revision rather than
BASE for the old target if the new target is explicitly specified, so that
it is the reverse of the diff from 'svn diff --old=BAR --new=FOO'.

Patch by: Alexey Neyman <stilor{_AT_}>

* subversion/svn/diff-cmd.c
  (svn_cl__diff): Change defaults as described and simplify the logic.

r1442645 | stsp | 2013-02-05 10:21:33 -0600 (Tue, 05 Feb 2013)

Tell people who pass mixed target types to 'svn diff' synopsis 1 that they
are probably looking for one of the '--old X --new Y' invocations.

Suggested by: Alexey Neyman

* subversion/svn/diff-cmd.c
  (svn_cl__diff): Wrap the error from svn_cl__assert_homogeneous_target_type()
   with a somewhat verbose but informative message.

r1442640 | stsp | 2013-02-05 10:03:49 -0600 (Tue, 05 Feb 2013)

Make 'svn diff PATH URL' and 'svn diff URL PATH', where exactly two targets
are given and one target is a URL, a new shorthand invocation syntax for
'svn diff --old PATH --new URL' and 'svn diff --old URL --new PATH',

These invocations used to error out with a "cannot mix repository and working
copy targets" before. User may intuitively try to diff a working copy to a URL
in the manner allowed by these new shorthands, so this is a usability
enhancement (principle of least surprise).

Suggested by: Alexey Neyman

* subversion/svn/diff-cmd.c
  (svn_cl__diff): Map a 2-target diff invocation where one target is a URL and
   the other target is a path to 'svn diff --old TARGET1 --new TARGET2'.

* subversion/svn/svn.c
  (svn_cl__cmd_table): Update 'svn help diff' output accordingly.

r1380697 | stsp | 2012-09-04 10:52:47 -0500 (Tue, 04 Sep 2012)

* subversion/libsvn_client/diff.c:
  (diff_content_changed): Mark path as visited if diffed by external tool.
   Fixes duplicate occurrences of Index: lines.

Patch by: Alexey Neyman <>

r1239553 | stsp | 2012-02-02 05:35:09 -0600 (Thu, 02 Feb 2012)

Add --no-diff-properties and --patch options to 'svn diff'.
The --patch option implies --no-diff-properties and --show-copies-as-adds.

These options are useful when creating patches for consumers that
do not understand or do not care about Subversion properties.

Patch by: Alexey Neyman <>

* subversion/svn/cl.h
* subversion/svn/main.c
  New options, --no-diff-properties and --patch for svn diff.

* subversion/include/svn_client.h
  (svn_client_diff6,svn_client_diff_peg6): New argument, ignore_prop_diff.
  (svn_client_diff5,svn_client_diff_peg5): Update comments.

* subversion/libsvn_client/deprecated.c
  (svn_client_diff5,svn_client_diff_peg5): Pass FALSE as ignore_prop_diff.

* subversion/libsvn_client/diff.c
  (diff_cmd_baton): New field, ignore_prop_diff.
  (diff_props_changed): Do nothing if diff_cmd_baton->ignore_prop_diff is set.
  (svn_client_diff6,svn_client_diff_peg6): Pass ignore_prop_diff downstream
   via diff_cmd_baton.

* subversion/svn/diff-cmd.c
  (svn_cl__diff): Handle --patch and --no-diff-properties.

* subversion/svn/log-cmd.c
  (log_entry_receiver): Request property changes from svn_client_diff6.

r984565 | cmpilato | 2010-08-11 15:08:29 -0500 (Wed, 11 Aug 2010)

Fix the type of structures returned in bindings from svn_fs_paths_changed2().

* subversion/include/svn_fs.h
  (svn_fs_paths_changed2): Rename the argument from changed_paths_p to
    changed_paths2_p, so that it's different from argument to

* subversion/bindings/swig/svn_fs.i
  (changed_paths2_p): New %hash_argout_typemap, denotes apr_hash_t containing

Patch by: Alexey Neyman <stilor{_AT_}>
          (Tweaked by me.)

r927222 | rdonch | 2010-03-24 18:29:20 -0500 (Wed, 24 Mar 2010)

SWIG/Python: enable callback functions to return Subversion errors (by
throwing a SubversionException).

* subversion/bindings/swig/python/libsvn_swig_py/swigutil_py.c:
  (callback_exception_error): Extend to check for a SubversionException
   and attempt to translate it into an svn_error_t*.

* subversion/bindings/swig/python/tests/
  (SubversionRepositoryTestCase.test_cease_invocation): New test for
   the above functionality.

Patch by: Alexey Neyman <>
Review by: Jon Foster <>

r917640 | cmpilato | 2010-03-01 12:46:05 -0600 (Mon, 01 Mar 2010)

* subversion/svnlook/main.c
  (do_plist): Correctly handle invocations of 'svnlook plist --revprop' when
    used with '-t TXN_NAME' instead of '-r REV'.

Reported by: Alexey Neyman <stilor{_AT_}>