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

Re: [RFC] Issue 2269 - svn st|up Do Not Warn When Ran Against a Nonexistent Path

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Wed, 27 Aug 2008 11:01:19 +0100

On Tue, 2008-08-26 at 20:35 -0600, Jeremy Whitlock wrote:
> Hi All,
> As I was going over issue 2269, I realize that there were a few
> edge cases that kept me from submitting my patch for submission that
> might not be issues after all. Basically, the patch attached to issue
> 2269 handles the simple scenarios:
>
> svn st NONEXISTENT_PATH
> svn up NONEXISTENT_PATH
>
> The reason I didn't submit it for review was that for some reason, I
> was worried about the following two scenarios:
>
> 1) svn st -u NONEXISTENT_PATH (Where the path doesn't exist in the WC
> but does exist in the repository.)
> 2) svn up NONEXISTENT_PATH (Where the path doesn't exist in the WC but
> does exist in the repository.)
>
> The more I think about this now, I think both of these should be
> handled like any other non-existent path, just like svn rm which
> outputs an error:
>
> svn: 'NONEXISTENT_PATH' does not exist

In principle, we could have made Subversion treat those cases as an
error. However, until now it has treated both of these cases as
enquiring about an item that might (later) exist at that path, and I
don't think we can change that now without making quite a few users
unhappy.

> If that is the case, I could go back over the patch, update it for bit
> rot, recreate and submit with minor changes. If this patch should
> indeed handle the edge cases differently, there is still more work to
> do. Either way, I'm working on this issue again and would love some
> feedback so I can wrap it up. Here is a direct link to the issue for
> those interested in reading the issue comments:
>
> http://subversion.tigris.org/issues/show_bug.cgi?id=2269

It would be really helpful if you would include a log message with every
patch, even tentative patches, to help readers understand what the
changes are accomplishing.

Thanks.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-27 12:01:45 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.