Hi,
Kannan R wrote:
> On Wednesday 07 October 2009 04:17 PM, Kannan wrote:
>> Stefan Sperling wrote:
>
>>> Does this patch fix this case, too?
>
>
> [..]
>
> /me digs from the list of yet-to-be-committed patches.
I ve attached a new patch which fixes this by calling
svn_ra_neon__maybe_store_auth_info() once, instead of the repeated calls as
cmpilato has pointed out. Kannan, can you confirm this solves the issue you
see? If so I shall commit the patch.
[[[
While working on a wc of a repo which needs authentication, 'store
password unencrypted' prompt is getting invoked only for those svn
commands that access the repo URL or that have an URL as argument
(eg. svn ls http://localhost/repo) and not for wc oriented commands
such as `svn ls' or `svn cat -r 5 test.c' which also require
authentication, which is solved here. This also fixes redundant
invokation of the function 'svn_ra_neon__maybe_store_auth_info' at
various places.
* subversion/libsvn_ra_neon/util.c
(svn_ra_neon__request_create): Store auth credentials if we can.
* subversion/libsvn_ra_neon/session.c
(svn_ra_neon__do_get_uuid): Remove redundant invocation.
* subversion/libsvn_ra_neon/commit.c
(commit_close_edit): Remove redundant invocation.
* subversion/libsvn_ra_neon/fetch.c
(svn_ra_neon__get_latest_revnum, reporter_finish_report): Remove
redundant invocation.
]]]
Thank You.
--
Senthil Kumaran S
http://www.stylesen.org/
Received on 2010-08-06 14:53:24 CEST