Hmm.  I'm wondering if something changed with svn_config_read_auth somewhere...

You can just tweak svn_get_auth to return (None, None) for a quick fix, but this is a
bug that should be fixed

This is the doc string from svn_config_read_auth_data
"""
    svn_config_read_auth_data(apr_hash_t hash, char cred_kind, char realmstring, 
        char config_dir, apr_pool_t pool) -> svn_error_t
"""

Same problem on Ubuntu 7.10.

There was a patch to fix this posted on the hosting group a while back (pass a
apr_hash_t as the first argument) but I haven't been able to get it to work. It made
use of "csvn", which doesn't seem to be provided by Ubuntu in the any of the python
subversion bindings, or in the ctypes module. Here's the message anyway:

http://groups.google.com/group/google-code-hosting/browse_thread/thread/fe42b81d6864d9e3/eb667e69eca64612

Thanks for the pointer. 
Maybe the csvn is a typo? Because it only seems to be used in the import statement. 
I'll give this a try.

The patched script works for me (on OSX 10.5). 
It looks like the csvn import is supposed/expected to fail. 

Attached is the patch from that google groups message, since the one there is messed
up for copy-paste-patch by the formatting of the email. I'm not the patch author BTW,
credit to Zhang Le.

Indeed the csvn thing is a red herring. It is only their in lieu of the csvn bindings
being written by a summer of code project at some point in the future.

The proposal here is to do as fitz said in comment 2, plus pass the updated params to
the new svn bindings. Also a --password option is added as a workaround for the lack
of auth directory reading, so upload is possible without user intervention. e.g.
grep/awk values from ~/.subversion/auth/simple in a non portable way ;) and pass them
on to the upload program.

It might be nice if something like this (--password, extra try: except:) was added
"officially" to the support script, even if it were only to stop the current version
from crashing out on newer builds of the svn python bindings.

	0001-Add-password-option-as-a-workaround-for-changes-in-s.patch
	6.3 KB Download

(No comment was entered for this change.)

I have the same problem in Cygwin.

(No comment was entered for this change.)

I have the same problem on Debian testing.

(No comment was entered for this change.)

Problem persists, Ubuntu Hardy.

I have implemented a hackish solution to fix it under Ubuntu Hardy for my use-case.
Instructions are at:
http://www.mechanicalcat.net/richard/log/Python/Sane_Python_application_packaging__conclusion__for_now

Please take a look and see if you can reproduce this issue. 

(No comment was entered for this change.)

(No comment was entered for this change.)

Tony, can you review the change using the code review tool?
http://code.google.com/p/support/source/detail?spec=svn177&r=177

(No comment was entered for this change.)

This patch restores the svn bindings usage for accessing the authn cache and also
adds a fallback for systems which don't have the bindings or don't have a new enough
version (you need 1.5 or newer).

It also removes the command-line password option, which was introduced as a workaround.

	upload.diff
	6.3 KB Download

Andy, since you were reviewing this patch while it was being developed, can you
verify/submit this fix?

OK sure, there's still a thread about adding tests for the new functionality.  I'll
take care of this bug.