Return-Path: X-Original-To: apmail-subversion-dev-archive@minotaur.apache.org Delivered-To: apmail-subversion-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id A509C1742A for ; Mon, 11 May 2015 18:57:03 +0000 (UTC) Received: (qmail 50547 invoked by uid 500); 11 May 2015 18:57:03 -0000 Delivered-To: apmail-subversion-dev-archive@subversion.apache.org Received: (qmail 50504 invoked by uid 500); 11 May 2015 18:57:03 -0000 Mailing-List: contact dev-help@subversion.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list dev@subversion.apache.org Received: (qmail 50494 invoked by uid 99); 11 May 2015 18:57:03 -0000 Received: from Unknown (HELO spamd3-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 11 May 2015 18:57:03 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd3-us-west.apache.org (ASF Mail Server at spamd3-us-west.apache.org) with ESMTP id BCCC0182223 for ; Mon, 11 May 2015 18:57:02 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd3-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: -0.002 X-Spam-Level: X-Spam-Status: No, score=-0.002 tagged_above=-999 required=6.31 tests=[SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=disabled Received: from mx1-us-west.apache.org ([10.40.0.8]) by localhost (spamd3-us-west.apache.org [10.40.0.10]) (amavisd-new, port 10024) with ESMTP id LdMDeteaDD5c for ; Mon, 11 May 2015 18:57:01 +0000 (UTC) Received: from mail.brain.org (mail.brain.org [50.197.89.41]) by mx1-us-west.apache.org (ASF Mail Server at mx1-us-west.apache.org) with ESMTPS id C2D29230CB for ; Mon, 11 May 2015 18:57:01 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by mail.brain.org (Postfix) with ESMTP id AD20F179E0C1; Mon, 11 May 2015 11:56:54 -0700 (PDT) X-Virus-Scanned: Debian amavisd-new at fornix.brain.org Received: from mail.brain.org ([127.0.0.1]) by localhost (fornix.brain.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id GVXRi95UwRGa; Mon, 11 May 2015 11:56:54 -0700 (PDT) Received: from fmri.brain.org (fmri.brain.org [IPv6:2001:470:e966:5:223:dfff:fedf:433d]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.brain.org (Postfix) with ESMTPSA id 738FC179E0BC; Mon, 11 May 2015 11:56:54 -0700 (PDT) Message-ID: <5550FB76.805@reser.org> Date: Mon, 11 May 2015 11:56:54 -0700 From: Ben Reser User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Evgeny Kotkov , Subversion Development Subject: Re: Issue 4579 / svnmucc fails to process certain deletes References: In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit 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.