[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] commit --include-externals (v2)

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Mon, 07 Nov 2011 08:44:13 -0500

On 11/05/2011 05:42 AM, Stefan Sperling wrote:
> I don't think changing the current default behaviour is a good idea.
> Changing the default behaviour is more likely to break existing
> scripts than a new configuration option.


> Also, please do not get too side-tracked because of the discussion about
> having a permanent configuration option. It is a logical next step to
> follow up on your current patch. But it does in no way affect the quality
> and usefulness of your current patch. So let's get that in first, and worry
> about more tweaks and knobs later.

+1 again. Get the functionality into place, preserving the current default
behavior and enabling the new with --include-externals. As we (and others)
grow accustomed to using the feature, we can revisit the defaults and
extended configuration options later with more than just our whims to drive
our decisions.

C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand

Received on 2011-11-07 14:44:51 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.