Re: svn commit: rev 6953 - in trunk/subversion: include libsvn_wc tests/clients/cmdline

On Wed, Sep 03, 2003 at 11:12:46PM +0200, Sander Striker wrote:
> > From: Greg Stein [mailto:gstein@lyra.org]
> > Sent: Wednesday, September 03, 2003 8:41 PM
>
> [...]
> >> (svn_wc_revert): throw an error if trying to revert the current working
> >> directory.
> >
> > Euh... why is this? It seems like an improper "fix". Why can't the current
> > directory simply be unscheduled?
>
> There were two options, one was to throw an error if you are in a dir
> you want to unschedule, the other was to walk up to the parent and unschedule
> it there. Karl and I both thought that throwing the error was a good
> solution since we have other commands that don't work from within the
> target either.

"the other was to walk up to the parent and unschedule it there" ... why?
Is there "add" scheduling information in the parent that needs to be
altered?

In any case, this is one of those cases where the software tells you
exactly how to correct it, so the question is, "why didn't you just take
those steps for me?"

Maybe call it an enhancement, but the current behavior is a bit strange.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org