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

Re: Local changes to externals (Was: Removing externals)

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Thu, 05 May 2011 22:03:00 +0200

On 05.05.2011 09:22, Hans-Emil Skogh wrote:
>>>>> It ignores all externals.
> ...
>> It ignores *local* changes to the svn:externals property by
>> default now. Only if you commit the changes, the default update
>> will also update the externals. If you want to have 'update'
>> respect your local modifications to the svn:externals property, you
>> have to do so in the "update to revision" dialog now.
>
> Oh! I think this is pretty bad. It really flies in the face of all
> test-before-you-commit practices. :-( If this change goes through I
> think I'll be seeing a lot more of broken commits, where externals
> are involved...
>
> Is there any way we can improve this on the TSVN-side? Either by
> making an update always honor the current WC state of all
> svn:externals properties (my preference), or by adding a checkbox to
> the edit svn:externals dialog that does the update automatically if
> the external is changed (hopefully default-checked..).

I'll add an option to change the default.
The current default is what the CL client will use as its default too,
because checking for local mods to external props takes a long time and
this was done for performance reasons.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2731349
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-05-05 22:03:06 CEST

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