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

Re: Patch: add svn_client_get_repos_path

From: Barry Scott <barry_at_barrys-emacs.org>
Date: 2005-04-01 00:33:20 CEST

On Mar 31, 2005, at 12:40, Peter N. Lundblad wrote:

> On Thu, 31 Mar 2005, Max Bowsher wrote:
>
>> Barry Scott wrote:
>> Patches require log messages, but more importantly, in what situation
>> would
>> this new function be worth it compared with calling
>> svn_ra_open/svn_ra_get_repos_root directly? (I'm not saying it's not
>> worth
>> it, just wanting to be sure it is before promising to create it.)
>>
> I'm not Berry, but I can understand that someone just wanting
> an RA session would be glad to avoid having to mess with RA callbacks
> and
> authentication stuff, when that's already handled by the client APIs.
> But

I'm not Berry either :-)

> instead of new special-case functions, maybe we should export some
> general
> purpose client function to open an RA session, such as
> svn_client__ra_session_from_path?

I used this function to test out the code fragment I posted. Making
this function
a supported interface would make me happy.

Peter would you be willing to turn svn_client__ra_session_from_path
into a
supported svn_client_ra_session_from_path?

I hope Max that you understand the reason for this being in either in
svn_client or
implementing Peters suggestion to make svn_client__ra_session_from_path
a supported API.
I was surprised by the suggestion that this obvious improvement needed
explaining.

Barry

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Apr 1 00:34:34 2005

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.