Re: [PATCH] Issue 1846, RA layer part

On Wed, 16 Jun 2004, Greg Hudson wrote:

> Whoops, just noticed this part:
>
> + /* Check if the server has the required capability. */
> + if (!svn_ra_svn_has_capability(conn, SVN_RA_SVN_CAP_GET_LOCATIONS))
> + return svn_error_create (SVN_ERR_RA_NOT_IMPLEMENTED, NULL,
> + _("Server does not implement the
> get-locations"
> + " command"));
>
> (and presumably a bunch of associated gunk elsewhere). I don't think
> there's a good reason to add a capability string for each new command.
> Just try the command and catch the resulting error if it's an old
> server.

Then I must have misunderstood our earlier discussion about detecting old
servers. Will fix.

Thanks,//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jun 16 22:05:01 2004