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

Re: [PATCH] svn status ^/ foo: The node '/path/to/wc/foo' was not found.

From: Uwe Stuehler <subversion-lists_at_bsdx.de>
Date: Mon, 14 Jun 2010 03:44:44 +0200

On 06/13/10 06:25 PM, Uwe Stuehler wrote:
> Index: subversion/libsvn_client/cmdline.c
> ===================================================================
> --- subversion/libsvn_client/cmdline.c (revision 953325)
> +++ subversion/libsvn_client/cmdline.c (working copy)
> @@ -123,6 +123,7 @@ check_root_url_of_target(const char **root_url,
> * argument.
> */
> if ((err->apr_err == SVN_ERR_ENTRY_NOT_FOUND)
> + || (err->apr_err == SVN_ERR_WC_PATH_NOT_FOUND)
> || (err->apr_err == SVN_ERR_WC_NOT_WORKING_COPY))
> {
> svn_error_clear(err);
> With this change svn_cl__status() actually makes it past the
> argument parsing phase:

Sorry, replying to myself since I forgot to include a log message.

Fix handling of non-WC paths in presence of repository relative URLs

* subversion/libsvn_client/cmdline.c:
   (check_root_url_of_target): Catch a new error constant for paths
   that are not registered in a working copy, and thereby allow the
   argument processing to continue as expected.

Patch by: Uwe Stuehler (subversion-lists_at_bsdx.de)

Actually, in the example that I used in my previous mail:

$ cd /path/to/wc
$ svn status ^/ foo

As far as I understood the purpose of check_root_url_of_target(),
it is applied in order to make sure that if ^/ is being used, all
other arguments refer to the same repository. Is that correct?

(That got me a bit confused, since I previously assumed that ^/
is always expanded to the repository URL of the current working
directory and not based on any other arguments that I supply.)

Even though 'foo' is not under SVN control and does not exist yet,
the argument points to a path *inside* the current working copy.
So I wonder: shouldn't it be possible to retrieve the repository
URL of that (nonexistent) path anyway? It would be possible to
infer this from the parent directory ("." in this case).

Received on 2010-06-14 03:45:46 CEST

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