You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Jack Repenning <ja...@tigris.org> on 2008/04/04 21:31:15 UTC
from main.c
> /* ### This really belongs in libsvn_client. The trouble is,
> there's no one place there to run it from, no
> svn_client_init(). We'd have to add it to all the public
> functions that a client might call. It's unmaintainable to do
> initialization from within libsvn_client itself, but it seems
> burdensome to demand that all clients call svn_client_init()
> before calling any other libsvn_client function... On the other
> hand, the alternative is effectively to demand that they call
> svn_config_ensure() instead, so maybe we should have a generic
> init function anyway. Thoughts? */
> err = svn_config_ensure(opt_state.config_dir, pool);
>
My thought: yes, there might as well / ought to be something like
svn_client_init(). Speaking as a client (SCPlugin) who's only today
figuring out that we have this obligation to call svn_config_ensure(),
but very likely would have looked for an init() routine years ago.
But I suppose I'm about five years too late on this one. Perhaps, at
least, a note in the "Using the APIs" chapter of The Book, or something?
By the way, does anyone know any *other* stuff I really ought to do,
being a libsvn_client client an' all?
-==-
Jack Repenning
jackrepenning@tigris.org
Project Owner
SCPlugin
http://scplugin.tigris.org
"Subversion for the rest of OS X"
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Re: Adding svn_client_init() (was: from main.c)
Posted by Jack Repenning <ja...@tigris.org>.
On Apr 4, 2008, at 2:47 PM, Karl Fogel wrote:
>
> Did you plug it in?
I tried, but it's so dark in here, what with the power being off, I'm
not sure I got the plug in the right hole.
-==-
Jack Repenning
jackrepenning@tigris.org
Project Owner
SCPlugin
http://scplugin.tigris.org
"Subversion for the rest of OS X"
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Adding svn_client_init() (was: from main.c)
Posted by Karl Fogel <kf...@red-bean.com>.
Jack Repenning <ja...@tigris.org> writes:
>> /* ### This really belongs in libsvn_client. The trouble is,
>> there's no one place there to run it from, no
>> svn_client_init(). We'd have to add it to all the public
>> functions that a client might call. It's unmaintainable to do
>> initialization from within libsvn_client itself, but it seems
>> burdensome to demand that all clients call svn_client_init()
>> before calling any other libsvn_client function... On the other
>> hand, the alternative is effectively to demand that they call
>> svn_config_ensure() instead, so maybe we should have a generic
>> init function anyway. Thoughts? */
>> err = svn_config_ensure(opt_state.config_dir, pool);
>
> My thought: yes, there might as well / ought to be something like
> svn_client_init(). Speaking as a client (SCPlugin) who's only today
> figuring out that we have this obligation to call svn_config_ensure(),
> but very likely would have looked for an init() routine years ago.
>
> But I suppose I'm about five years too late on this one. Perhaps, at
> least, a note in the "Using the APIs" chapter of The Book, or
> something?
There is no reason we couldn't add svn_client_init() now, and call
svn_client_init() from it, and document both so that the history here is
clear, and make the change you suggest to the book and wherever else.
Anyone object?
> By the way, does anyone know any *other* stuff I really ought to do,
> being a libsvn_client client an' all?
Did you plug it in?
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org