[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[sparse directories] Non-recursive mode and 'info', 'propset', etc. APIs (was Re: svn commit: r26752 ...)

From: Daniel L. Rall <dlr_at_collab.net>
Date: 2007-09-24 22:24:52 CEST

On Sat, 22 Sep 2007, Hyrum K. Wright wrote:

> hwright@tigris.org wrote:
...
> > Log:
> > Extend depth parameter from svn_client_propset3() to JavaHL. Followup to
> > r26745.
> >
> > (Note: This adds another JavaHL test failure to the one already present.)
>
> This test failure was present even if I just use the
> SVN_DEPTH_TO_RECURSE macro in SVNClient::propertySet() instead of
> pushing the depth parameter all the way through to the Java API. I
> decided that getting JavaHL to build was of higher priority than
> immediately diagnosing the failing test.
...
> testOODStatus(org.tigris.subversion.javahl.BasicTests)junit.framework.AssertionFailedError:
> commit item for
> file:///home/hwright/dev/svn-apache/subversion/bindings/javahl/test-work/repositories/basic_test4/iota
> not expected
 
After discussing this with Hyrum, I fixed this problem today in r26766.

This was caused by use of Depth.fromRecurse(), Java code with the same
behavior as that of the SVN_DEPTH_FROM_RECURSE() macro. Both the 'propset'
and 'info' APIs were making use of this type of conversion, which maps
non-recursive modes to svn_depth_files, breaking backwards compatibility.
APIs of this type should instead map non-recursive to svn_depth_empty.

Are there more cases of this problem?

  • application/pgp-signature attachment: stored
Received on Mon Sep 24 22:25:08 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.