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

Re: svn commit: r1329015 - in /subversion/trunk/subversion: include/private/svn_ra_private.h libsvn_ra/editor.c libsvn_ra/ra_loader.c libsvn_ra/ra_loader.h

From: Hyrum K Wright <hyrum.wright_at_wandisco.com>
Date: Tue, 24 Apr 2012 08:59:57 -0500

On Mon, Apr 23, 2012 at 9:47 PM, Greg Stein <gstein_at_gmail.com> wrote:
>
> On Apr 23, 2012 7:20 PM, "Hyrum K Wright" <hyrum.wright_at_wandisco.com> wrote:
>>
>> On Mon, Apr 23, 2012 at 4:55 PM, Greg Stein <gstein_at_gmail.com> wrote:
>> > On Mon, Apr 23, 2012 at 16:21, Hyrum K Wright
>> > <hyrum.wright_at_wandisco.com> wrote:
>> >> On Mon, Apr 23, 2012 at 12:52 PM, Greg Stein <gstein_at_gmail.com> wrote:
>> >>...
>> >>> Alright. It sounds like svn_ra.h has two purposes: stuff for
>> >>> libsvn_client,
>> >>> and stuff for all RA layers to use internally. Ugh.
>> >>
>> >> But the capability strings are part of the public API.
>> >
>> > Says who? Not server Ev2 support. The RA layer can isolate that, so it
>> > should not be "part of the public API".
>> >
>> > Mergeinfo support? Sure. But Ev2? Nah.
>>
>> The entire RA interface is part of the public API.  The wire protocols
>> are the public interface to the server, particularly for those who
>> want to write their own clients and server.  These strings are part of
>> that definition.
>
> Ah. There we go. That clinches it.
>
> It's too bad we don't have svn_wire.h. Right now, svn_ra serves both
> purposes, so yah... I guess it makes sense to leave it there, unless we'd
> like to start migrating appropriate pieces to a new header. Thoughts?

I don't have a whole lot of opinions on the subject, as it is largely
outside the scope of Ev2.

-Hyrum

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com/
Received on 2012-04-24 16:00:32 CEST

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.