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

RE: [PATCH] Provide meaningful message when svn update is run against an out of date mirror.

From: Kamesh Jayachandran <kamesh_at_collab.net>
Date: Sat, 10 Oct 2009 00:15:16 +0530

>I'm not fond at all of the idea of having code that is completely ignorant
>of the fact that a proxy is or isn't in place (like mod_dav_svn's update
>logic) generating errors that refer to mirrors.

I agree with you may be it can refer for the existence of the SVNMasterURI directive and and trigger this error?

What do you think?

>A more appropriate approach would be for the mirror.c code (which is the
>only code that really knows that some proxying is going on) to detect the
>generic error and replace it with a more specific one in the returned
>stream. That's probably quite a bit more work to achieve, but I believe it
>is far more correct that what you are proposing.

IIUC 'svn update' does not exercise the mirror code!

With regards
Kamesh Jayachandran

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2405638
Received on 2009-10-09 20:48:41 CEST

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