[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: Bert Huijben <bert_at_qqmail.nl>
Date: Fri, 4 Nov 2011 13:29:29 +0100

> -----Original Message-----
> From: C. Michael Pilato [mailto:cmpilato_at_collab.net]
> Sent: vrijdag 4 november 2011 12:31
> To: Neels J Hofmeyr
> Cc: Alexey Neyman; dev_at_subversion.apache.org
> Subject: Re: [PATCH] commit --include-externals (v2)
> On 11/04/2011 06:55 AM, Neels J Hofmeyr wrote:
> > Right now my focus is still on the "inner" behavior and so far it seems most
> > other devs have different things on their mind. I'm still gauging whether it
> > is permissive or dismissive silence, +0 or -0 ;)
> +0 from me. :-)

+0 on including the support to allow the commits. (I haven't reviewed the patch in detail yet, but the idea is good)

-0 to -0.5 on making it configurable as default behavior.

As project we try not to add many configuration options that change the default behavior of our client.

Making this configurable changes the behavior of scripts calling svn and makes it harder to test subversion by increasing the number of test scenarios.

> --
> C. Michael Pilato <cmpilato_at_collab.net>
> CollabNet <> www.collab.net <> Distributed Development On Demand
Received on 2011-11-04 13:30:08 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.