Julian Foad wrote:
> Senthil Kumaran S wrote:
>> Arfrever Frehtes Taifersar Arahesis wrote:
>>> svn_auth_get_platform_specific_client_providers() and
>>> svn_auth_get_platform_specific_provider() could accept additional
>>> 'svn_boolean_t command_line' argument. If 'command_line==TRUE', then
>>> svn_auth_get_gnome_keyring_simple_provider() would be called with
>>> unlock_prompt_func==NULL and prompt_baton==NULL.
>> That is a good one, I shall come up with a patch with the above idea :)
>
> Instead of this function having internal knowledge of a particular
> command-line prompting function, it would be better for it to take a
> pointer to a prompting function and have no internal knowledge. (The
> file "auth.c" should not have to include "svn_cmdline.h".)
Agreed. This approach is preferable.
-Hyrum
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1011826
Received on 2009-01-08 16:14:58 CET