On Thu, Mar 6, 2008 at 5:25 AM, Julian Foad <julianfoad_at_btopenworld.com> wrote:
> Code like this appears in several places:
>
> > err = editor->delete_entry(path, item->revision, parent_baton, pool);
> > if (err)
> > {
> > if (err->apr_err == SVN_ERR_FS_NOT_FOUND
> > || err->apr_err == SVN_ERR_RA_DAV_PATH_NOT_FOUND)
>
> These two errors both mean "the item was not present in the repository". When
> committing to the repository, this indicates that the WC is out of date w.r.t.
> the repository.
>
> SVN_ERR_FS_NOT_FOUND is sent by the file or svn RA layers, and
> SVN_ERR_RA_DAV_PATH_NOT_FOUND is sent by the http/https RA layers, with the
> same meaning.
Julian,
Thanks, this reminded me of a similar case a couple of months ago --
which I subsequently forgot about :-(
There the problem is with
svn_ra_get_locations(svn_ra_session_t *session,
apr_hash_t **locations,
const char *path,
svn_revnum_t peg_revision,
apr_array_header_t *location_revisions,
apr_pool_t *pool)
If PATH doesn't exist at PEG_REVISION:
RA_LOCAL/RA_SVN: Returns SVN_ERR_FS_NOT_FOUND
RA_NEON: Returns two SVN_ERR_RA_DAV_PATH_NOT_FOUND errors
RA_SERF: Returns SVN_NO_ERROR, sets *LOCATIONS to an empty hash.
See: http://svn.haxx.se/dev/archive-2007-12/0457.shtml. Anyhow, I
bring this up mainly because in that thread Karl suggests, "The
fix(es) should happen as close to the original (server-side) source of
the error as possible, IMHO." Barring that, the fix should be in the
particular RA implementation.
When you say the "RA layers ought to be made to return the same error
code for this case" which approach were you thinking of, a fix as much
on the server side as possible or strictly within the RA
implementations?
Paul
> The RA layers are intended to provide a consistent interface, so I think all
> the RA layers ought to be made to return the same error code for this case.
>
> At the very least, svn_delta_editor_t ought to document the error codes that
> its functions return in this and similar situations.
>
> Opinions?
>
> - Julian
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
> For additional commands, e-mail: dev-help_at_subversion.tigris.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-03-06 17:15:45 CET