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

Re: Relocate warning dialog is broken

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Fri, 06 May 2011 19:30:50 +0200

On 06.05.2011 18:15, Nikolay Osinin wrote:
>> On 28.04.2011 13:23, Nikolay Osinin wrote:
>>>>> 1. Why TSVN warns me when I relocating WC from one to another repo?
>>>>> Here is full URLs. Repo URL in round bracket.
>>>>> From: (https://topserver01:8010/svn/TestCentral)/Gal/trunk/CompSrc/M/M_Transp
>>>>> To: (https://topserver01:8010/svn/Central)/Gal/trunk/CompSrc/M/M_Transp
>>>>
>>>> Because by just looking at the urls, TSVN doesn't know that TestCentral
>>>> and Central are two different repositories. 'svn' could be the
>>>> repository root.
>>>> That's also why the message says "it seems" and not "it definitely is".
>>>>
>>>
>>> May be it is possible to detect repository root by URL?
>>> I tried to do this in the patch.
>>
>> The problem with your patch:
>> * the check must not access the repository
>> * in a real relocate, the source repository wouldn't be accessible
>> anymore, so contacting that repo to find the repo root wouldn't even work
>>
>> But thanks anyway :)
>
> Excuse me for my impudence.
> But I'd like to offer another one version of the patch.
> 1. If the source repository is not accessible then we can get the repo URL from the working copy info.
> 2. Why the check should not access the repository? Cmd-line tool svn.exe does.

That warning is only a warning. The relocate command when really
executed does checks that access the repository too. But if the
repository isn't available, the relocate will succeed anyway without any
warnings. So we show a warning beforehand.
What you're trying to do here is the same that the relocate command does
later (and some more).
But: as I mentioned before, in most situations where a relocate is done,
the repository isn't accessible anymore so the checks will fail.

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=2732153
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-05-06 19:30:57 CEST

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