Thanks Hyrum!
I committed the test and nominated all three revisions for 1.7.x.
On Tue, Aug 16, 2011 at 4:35 PM, Hyrum K Wright
<hyrum.wright_at_wandisco.com>wrote:
> r1158436 allows you to specify the base revision for the URL to
> satisfy the underlying API requirements. I didn't commit the test.
>
> (I assume that once the test and other fixes are in place, the whole
> lot will be nominated for 1.7.x.)
>
> -Hyrum
>
> On Tue, Aug 16, 2011 at 2:28 PM, Mark Phippard <markphip_at_gmail.com> wrote:
> > Attaching this time ...
> >
> > On Tue, Aug 16, 2011 at 3:26 PM, Mark Phippard <markphip_at_gmail.com>
> wrote:
> >>
> >> 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/
> >
> >
> >
> > --
> > Thanks
> >
> > Mark Phippard
> > http://markphip.blogspot.com/
> >
>
>
>
> --
>
> uberSVN: Apache Subversion Made Easy
> http://www.uberSVN.com/
>
--
Thanks
Mark Phippard
http://markphip.blogspot.com/
Received on 2011-08-16 23:55:35 CEST