You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@subversion.apache.org by ju...@apache.org on 2010/01/13 15:41:30 UTC

svn commit: r898777 - /subversion/trunk/subversion/libsvn_ra/ra_loader.c

Author: julianfoad
Date: Wed Jan 13 14:41:29 2010
New Revision: 898777

URL: http://svn.apache.org/viewvc?rev=898777&view=rev
Log:
On failure to connect to the repository, give a consistent and readable
error message. This particularly helps in cases where the error message
from the specific RA layer is poor, such as RA-Neon's "200 OK" error message.

* subversion/libsvn_ra/ra_loader.c
  (svn_ra_open3): Wrap any error returned from the open_session() method.

Modified:
    subversion/trunk/subversion/libsvn_ra/ra_loader.c

Modified: subversion/trunk/subversion/libsvn_ra/ra_loader.c
URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra/ra_loader.c?rev=898777&r1=898776&r2=898777&view=diff
==============================================================================
--- subversion/trunk/subversion/libsvn_ra/ra_loader.c (original)
+++ subversion/trunk/subversion/libsvn_ra/ra_loader.c Wed Jan 13 14:41:29 2010
@@ -482,8 +482,10 @@
   session->pool = pool;
 
   /* Ask the library to open the session. */
-  SVN_ERR(vtable->open_session(session, repos_URL, callbacks, callback_baton,
-                               config, pool));
+  SVN_ERR_W(vtable->open_session(session, repos_URL, callbacks, callback_baton,
+                               config, pool),
+            apr_psprintf(pool, "Unable to connect to a repository at URL '%s'",
+                         repos_URL));
 
   /* Check the UUID. */
   if (uuid)