Re: svn commit: rev 1007 - trunk/subversion/tests/clients/cmdline

Philip Martin <philip@codematters.co.uk> writes:

> > The best way to join URLs is with:
> >
> > import urlparse
> > url = urlparse.urljoin(svntest.main.test_area_url, svntest.main.current_repo_dir)
>
> Ah, OK. However as I don't really do Python I copied it from
> actions.py...
>
> I guess changing action.py as well will make
>
> -url http://localhost/
>
> work, at present it must be specified without a trailing slash
>
> -url http://localhost

Yes it does, but it breaks the tests over ra_local.

url = urlparse.urljoin(main.test_area_url, main.pristine_dir)
print main.test_area_url
print main.pristine_dir
print url

gives

file:///home/pm/sw/subversion/obj/subversion/tests/clients/cmdline
local_tmp/repos
file:/home/pm/sw/subversion/obj/subversion/tests/clients/local_tmp/repos

Not only has it stripped the 'cmdline' part but it has converted
'file:///' into 'file:/'. Subversion then fails with

 svn_ra_local__split_URL: URL does not contain `file://' prefix

Now if I ensure that the first bit is slash terminated I avoid the
cmdline stripping, but I can't get around the file:/// to file:/
conversion.

-- 
Philip
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org