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

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.

Agreed.

> 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