Re: svn commit: r1655474 - /subversion/trunk/subversion/tests/cmdline/redirect_tests.py

Philip Martin <philip.martin_at_wandisco.com> writes:

> E175011 has the wrong name, the DAV layer knows nothing about relocation
> which is a libsvn_client concept so SVN_ERR_RA_DAV_RELOCATED should have
> been SVN_ERR_RA_DAV_REDIRECTED. It's too late to change that now but I
> suppose we could change the error text and add SVN_ERR_RA_DAV_REDIRECTED
> as an alias for SVN_ERR_RA_DAV_RELOCATED.

The current error message is poor, using trunk:

$ svn st -u wc
svn: E170013: Unable to connect to a repository at URL 'http://localhost:8888/obj/PERM/repo'
svn: E175011: Repository moved permanently to 'http://localhost:8888/obj/repo'; please relocate

and using 1.8:

$ svn st -u wc
svn: E175011: Repository moved permanently to 'http://localhost:8888/obj/repo'; please relocate

The "please relocate" is added by ra_serf and that's the wrong place to
be recommending the use of wc/client feature. It's also not necessarily
the best advice, an update is much simpler and will handle things
automatically. The user would only prefer relocate when there was some
reason to avoid an update.

-- 
Philip Martin | Subversion Committer
WANdisco // *Non-Stop Data*