Re: 1.8.0-rc1 up for testing/signing

On Fri, May 10, 2013 at 12:33 PM, Branko Čibej <brane_at_wandisco.com> wrote:
> On 10.05.2013 18:22, Mark Phippard wrote:
>> On Fri, May 10, 2013 at 11:42 AM, Mark Phippard <markphip_at_gmail.com> wrote:
>>> On Wed, May 8, 2013 at 12:40 PM, Mark Phippard <markphip_at_gmail.com> wrote:
>>>> I am using a new build process and using VS 2012, so I am sure that is
>>>> the problem. But I am getting 49 test failures. I am attaching the
>>>> fails.log but I do not think it lends much ideas. I tried running
>>>> with --verbose but the output is the same.
>>>>
>>>> Getting the same failures over all three ra layers.
>>> Trying to figure out what is wrong, so setup my PATH to use the 1.8.0
>>> binaries and run svnsync manually. Maybe my brain is just not
>>> working, but what did I do wrong here?
>>>
>>> C:\svn-1.8>svnsync init file:///svn-1.8/repos https://hostname/svn/test
>>> svnsync: E000022: Session is rooted at 'file:///svn-1.8/repos' but the
>>> repos root is 'file:///C:/svn-1.8/repos'
>>>
>>> Basically just trying to sync a small remote repos to a local copy.
>>>
>>> Since it is the svnsync and svnrdump tests that were failing, that is
>>> what I tested. I cannot see why I would get a bug like this and
>>> others would not, so I am assuming I am just not functioning right
>>> today.
>> I think this might be an actual bug on Windows, but do not see why
>> others do not get it. Note how the error message changed the URL I
>> entered to something different. Perhaps some kind of canonicalization
>> issue?
>>
>> I have tried other variants of that file:// URL but none work for me.
>>
>> I suspect an error like this is why all my svnsync and svnrdump tests
>> are failing. That said, I cannot do a simple command like this
>> either:
>>
>> svn ls file:///svn-1.8/repos
>>
>> That shows the same sort of problem where it is changing the URL.
>> What is going on? My tests passed for most cases, so there must be
>> some way of correctly specifying a file:// URL on Windows that is
>> working.
>
> The correct way includes the drive letter:
>
> svn ls file:///C:/svn-1.8/repos
>
> And I expect you need the same in 'svnsync init' as well. Could be that
> the test driver doesn't get that right -- although I'd wonder how it
> worked in the past then; it's always been this way.

Thanks, that works and so did svnsync. Kind of too bad though,
because it does not help me in terms of knowing why the tests are
failing.

FWIW, I do not agree with you on the syntax though.

file:///foo

Has always worked on Windows to access C:\foo

The drive letter was only needed to access other drives. This is a
behavior change that either needs to be fixed or documented. It will
absolutely break things on Windows.

--
Thanks
Mark Phippard
http://markphip.blogspot.com/