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

Re: svn commit: r966774 - /subversion/branches/issue-2779-dev/subversion/libsvn_client/update.c

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Thu, 22 Jul 2010 14:12:50 -0400

On 07/22/2010 02:09 PM, C. Michael Pilato wrote:
> On 07/22/2010 01:59 PM, Bert Huijben wrote:
>>> -----Original Message-----
>>> From: cmpilato_at_apache.org [mailto:cmpilato_at_apache.org]
>>> Sent: donderdag 22 juli 2010 19:55
>>> To: commits_at_subversion.apache.org
>>> Subject: svn commit: r966774 - /subversion/branches/issue-2779-
>>> dev/subversion/libsvn_client/update.c
>>>
>>> Author: cmpilato
>>> Date: Thu Jul 22 17:54:49 2010
>>> New Revision: 966774
>>>
>>> URL: http://svn.apache.org/viewvc?rev=966774&view=rev
>>> Log:
>>> On the 'issue-2779-dev' branch: Get 'svn update' following redirects!
>
> [...]
>
>> Shouldn't we relocate the complete working copy, starting by the working
>> copy root instead of just the operation anchor?
>>
>> This gives you a partly relocated working copy without proper
>> notifications to the user. (In AnkhSVN there are a lot of users that
>> update some part of their working copy. I would assume that this also
>> applies to other clients)
>
> Ooh! Good catch, Bert!
>
> (No notifications have been added yet. I believe they should be present in
> all cases, working copy root or otherwise. I just haven't coded them yet.)

Hrm... 'svn relocate' just calls svn_client_relocate() as well. I don't
see any calls to svn_wc_is_wc_rootX() or anything. Does that allow partial
relocations, too?

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

Received on 2010-07-22 20:13:27 CEST

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