subversion-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ben Reser <...@reser.org>
Subject Re: Issue 4579 / svnmucc fails to process certain deletes
Date Mon, 11 May 2015 18:56:54 GMT
On 5/11/15 11:41 AM, Evgeny Kotkov wrote:
> I raised an issue because I think that we care about compatibility and about
> *not* breaking user scripts and tools that happen to use 'svnmucc'.  Other
> possible examples that work in 1.8.13, but fail in 1.9.0 RC1 include:
> 
>   svnmucc rm /A/mu rm /A
> 
>   svnmucc cp HEAD /iota /A/iota rm /A
> 
> If this behavior change isn't a regression, I am fine with closing the issue.

The first example seems like a much better example, because that probably
should be allowed, even though it's redundant.

But the second one is still an example of where I think the user is poorly
served by us allowing it because we're throwing a change the user desired to be
applied.

The cp example here isn't such a huge issue because /iota is still there, but
the put examples are potentially a situation of data loss if the source file
they are putting is destroyed.

If the cost of helping users avoid inadvertent mistakes is disallowing the rm
example above and breaking some scripts then I'm on the side of some slight
incompatibility here.

While we certainly have endeavored to avoid gratuitous changes in command line
behavior we have made changes in the past with good reason.  I don't know if
this behavior change was deliberate or not.  But in my opinion it's probably
for the best.


Mime
View raw message