[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Thu, 29 Jan 2015 12:27:48 +0000

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*
Received on 2015-01-29 13:30:36 CET

This is an archived mail posted to the Subversion Dev mailing list.