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

Re: JavaHL Bug: cannot propedit versioned props by URL

From: Mark Phippard <markphip_at_gmail.com>
Date: Tue, 16 Aug 2011 15:26:01 -0400

On Tue, Aug 16, 2011 at 3:09 PM, Hyrum K Wright
<hyrum.wright_at_wandisco.com>wrote:

> On Tue, Aug 16, 2011 at 11:59 AM, Mark Phippard <markphip_at_gmail.com>
> wrote:
> > The JavaHL propertySetRemote API seems incomplete.
> > 1) It does not take a CommitMessageCallback. So no way to provide commit
> > message.
>
> r1158421 added the CommitMessageCallback to the propertSetRemote API.
>
> > 2) When trying to change a versioned property via URL, it fails with:
> > Bogus revision information given
> > svn: Setting property on non-local targets needs a base revision
> > We want to use this API in Subclipse to freeze svn:externals properties
> in a
> > tag after committing it. TortoiseSVN seems to offer to do this now.
>
> I'm not sure exactly what's going on here, or how to trigger it. If
> possible, a test case would go a long way toward illuminating this.
>

I have attached a patch that adds a new test that shows the problem. When
this is working I would probably enhance the test to update the WC and
verify the prop change.

When I run the test with current trunk it fails with this:

There was 1 error:
1)
testPropEdit(org.apache.subversion.javahl.BasicTests)org.apache.subversion.javahl.ClientException:
Bogus revision information given
svn: Setting property on non-local targets needs a base revision

at native.subversion.libsvn_client(prop_commands.c:418)
at org.apache.subversion.javahl.SVNClient.propertySetRemote(Native Method)
at
org.apache.subversion.javahl.BasicTests.testPropEdit(BasicTests.java:3146)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at org.apache.subversion.javahl.RunTests.main(RunTests.java:116)

FAILURES!!!
Tests run: 1, Failures: 0, Errors: 1

-- 
Thanks
Mark Phippard
http://markphip.blogspot.com/
Received on 2011-08-16 21:26:32 CEST

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