Re: [PATCH] Add svn_ra_initialize

kfogel@collab.net writes:

> "Peter N. Lundblad" <peter@famlundblad.se> writes:
>> Short summary: I'm crazy.
>>
>> Longer: I want to add an API to 1.2 that does nothing. That's the
>> svn_ra_initialize I mentioned in the thread about the fSFS fix. We will
>> need it in 1.3 to properly support calling svn_fs_initialize from the
>> client. Would it be accepted into 1.2?
>
> Maybe I need the _really_ long version...
>
> What's the purpose of putting it into 1.2, when we are allowed to add
> it in 1.3 anyway?

If svn_ra_initialize is present in 1.2 then we can require that it be
called before any other "new in 1.2" function. Most of the svn_ra_xxx
stuff happens to be new in 1.2, we deprecated the vtable approach.

I'm not sure why the patch doesn't implement svn_ra_initialize, just a
lack of time perhaps? If the --enable-dso problem is considered to be
sufficiently important to block the release then we have to implement
something, and it may as well be svn_ra_initialize.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org