Return-Path: X-Original-To: apmail-subversion-users-archive@minotaur.apache.org Delivered-To: apmail-subversion-users-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 1AC95E5C9 for ; Sat, 2 Feb 2013 13:08:21 +0000 (UTC) Received: (qmail 68299 invoked by uid 500); 2 Feb 2013 13:08:21 -0000 Delivered-To: apmail-subversion-users-archive@subversion.apache.org Received: (qmail 67882 invoked by uid 500); 2 Feb 2013 13:08:16 -0000 Mailing-List: contact users-help@subversion.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list users@subversion.apache.org Received: (qmail 67857 invoked by uid 99); 2 Feb 2013 13:08:15 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 02 Feb 2013 13:08:15 +0000 X-ASF-Spam-Status: No, hits=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: local policy) Received: from [217.140.74.2] (HELO continuum.iocl.org) (217.140.74.2) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 02 Feb 2013 13:08:09 +0000 Received: (from krey@localhost) by continuum.iocl.org (8.11.3/8.9.3) id r12D7ev04159; Sat, 2 Feb 2013 14:07:40 +0100 Date: Sat, 2 Feb 2013 14:07:39 +0100 From: Andreas Krey To: Daniel Shahaf Cc: Matt Hargett , "users@subversion.apache.org" Subject: Re: UNS: Re: adding --include-externals to svn diff Message-ID: <20130202130739.GC13329@inner.h.apk.li> References: <20130202095115.GC3518@lp-shahaf.local> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20130202095115.GC3518@lp-shahaf.local> User-Agent: Mutt/1.4.2.1i X-message-flag: What did you expect to see here? X-Virus-Checked: Checked by ClamAV on apache.org On Sat, 02 Feb 2013 11:51:15 +0000, Daniel Shahaf wrote: > Matt Hargett wrote on Sat, Feb 02, 2013 at 02:04:06 +0000: > > To parallel the additions of --include-externals to the 'commit' and 'ls' > > commands, I would also like to propose adding the option to the 'diff' > > command. I just tested latest trunk and the option is unrecognized. > ... > > Let me know what I can do, within reason, to help bring make these > > commands more congruent for 1.8.0. > > The first step is to define the behaviour of the would-be-added option. > For example, is it only valid for working copy targets? How does it > behave when one or both targets is a URL? Why, as if the URL were checked out and the diff done against the resulting tree (minus .svn dirs, o/c). Which causes the interesting thing that, when an external definition changes (like, from whatever/tags/3.0 to whatever/tags/3.1), you'd see the complete diff between 3.0 and 3.1. And the more interesting question whether the diff of the svn:external property itself should be shown at all. (I find it much more complicated to define what commit --include-externals would do - any one of 'leave the svn:externals alone' to -- when the svn:externals references trunk or branches, 'update the peg revs in the external' -- when using peg revisions, 'commit into the trunk instead of the referenced tag, create a new tag, and put that into the external' -- our usual workflow, except that we don't just blindly commit into the externals look desirable.) Andreas -- "Totally trivial. Famous last words." From: Linus Torvalds Date: Fri, 22 Jan 2010 07:29:21 -0800