[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: Fri, 04 Nov 2011 09:15:17 -0400

On 11/04/2011 09:13 AM, Mark Phippard wrote:
> On Fri, Nov 4, 2011 at 9:09 AM, C. Michael Pilato <cmpilato_at_collab.net
> <mailto:cmpilato_at_collab.net>> wrote:
>
> On 11/04/2011 08:36 AM, Mark Phippard wrote:
> > I can comment from a GUI tool perspective. In Subclipse, we have
> always had
> > this feature.
>
> [...]
>
> > FWIW, I think that the command line client should have always included
> > externals by default. I guess we cannot do that now, so having an
> option is
> > good. I kind of agree with Bert that I do not like the idea of making it
> > configurable.
>
> I'm confused, because my understanding was that the underlying commit APIs
> were not able to handle multi-working-copy commits (which would include
> externals) simultaneously until 1.7 because of code's inability to manage
> the ADM locks properly.
>
>
> I think we had this conversation before and you mentioned having put most of
> the code in place prior to 1.0 and it just was not finished. The way SVN
> worked prior to 1.7 was that it was very easy to trick it. As long as the
> common parent of a commit operation looked like a WC, then commit worked.
> So in the case, of externals, it simply just worked. But in Subclipse we
> were also to trick SVN into doing a single commit across multiple working
> copies by dynamically creating a fake .svn folder in the parent of the
> working copies.

Right right right. I do recall this now. Thanks for the mental nudge.

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

Received on 2011-11-04 14:15:59 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.