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