On Thu, Apr 1, 2010 at 6:07 PM, Paul Burba <ptburba_at_gmail.com> wrote:
> On Thu, Apr 1, 2010 at 9:50 AM, Philip Martin
> <philip.martin_at_wandisco.com> wrote:
>> Philip Martin <philip.martin_at_wandisco.com> writes:
>>
>>> Paul Burba <ptburba_at_gmail.com> writes:
>>>
>>>>> My bad, I didn't pass --url! Â I was still half asleep when I did that.
>>>>> Â The tests pass for me over ra_neon, but all except 77 fail over
>>>>> ra_serf.
>>>>
>>>> All 5 corresponding tests (45, 76, 78, 123, 125 for those playing at
>>>> home) pass on trunk [fsfs | ra_serf]
>>>
>>> Agreed. I get the same 4 failures as you, and they pass on trunk.
>>
>> This is the revision that causes the failures:
>>
>> r923779 | hwright | 2010-03-16 14:48:28 +0000 (Tue, 16 Mar 2010) | 16 lines
>>
>> Reintegrate the 1.6.x-issue-3242-partial branch:
>>
>> Â * ^/subversion/branches/1.6.x-issue-3242-partial
>> Â Merge a portion of the issue 3242 work, specifically the bit that
>> Â makes mergeinfo queries in the client layer happen against the
>> Â current RA session URL instead of forcibly against the repository
>> Â root URL (which might not be readable by the user per authz
>> Â policy). Â While this is not a full solution to issue #3242, it
>> Â should get us back to Subversion 1.4-level behavior which has
>> Â pretty consistently been deemed acceptable (enough) by our user
>> Â base.
>> Â Justification:
>> Â Â Issue #3242 is a hot issue. Â Hot topic, hot tempers, etc.
>> Â Votes:
>> Â Â +1: cmpilato, ivan, rhuijben
>
> This is also the revision that is causing the failure with
> basic_tests.py 37 "use folders with names like 'c:hi'".
>
> With 1.6.x-issue-3242-partial in place, basic_tests.py 37 now calls
> svn_client__open_ra_session_internal(), in
> libsvn_client/delete.c:delete_urls(), on the specific URL to be
> deleted:
>
> "file:///C%3A/SVN/src-branch-1.6.x/Debug/subversion/tests/cmdline/svn-test-work/repositories/basic_tests-37/c:hi"
>
> rather than the root of the reposisotry:
>
> "file:///C%3A/SVN/src-branch-1.6.x/Debug/subversion/tests/cmdline/svn-test-work/repositories/basic_tests-37"
>
> Note the trailing part now in the URL: "/c:hi". Â This ultimately
> causes svn_io_check_path() in libsvn_repos/repos.c:check_repos_path()
> to fail because it can't handle the ':'
>
> Â Â Â Â libsvn_repos-1.dll!check_repos_path() Â Line 1265 Â Â Â Â C
> Â Â Â Â libsvn_repos-1.dll!svn_repos_find_root_path() Â Line 1366 Â Â Â Â C
> Â Â Â Â libsvn_ra-1.dll!svn_ra_local__split_URL() Â Line 131 Â Â C
> Â Â Â Â libsvn_ra-1.dll!svn_ra_local__open() Â Line 454 Â C
> Â Â Â Â libsvn_ra-1.dll!svn_ra_open3() Â Line 481 Â Â Â Â C
> Â Â Â Â libsvn_client-1.dll!svn_client__open_ra_session_internal() Â Line 296 Â Â C
> Â Â Â Â libsvn_client-1.dll!delete_urls() Â Line 187 Â Â C
> Â Â Â Â libsvn_client-1.dll!svn_client_delete3() Â Line 270 Â Â Â C
> Â Â Â Â svn.exe!svn_cl__delete() Â Line 83 Â Â Â C
> Â Â Â Â svn.exe!main() Â Line 2119 Â Â Â C
> Â Â Â Â svn.exe!__tmainCRTStartup() Â Line 582 Â C
> Â Â Â Â svn.exe!mainCRTStartup() Â Line 399 Â Â Â C
>
> There are a slew of fixes in this space on trunk that will probably
> need to be backported,
>
> Â http://svn.apache.org/viewvc?view=revision&revision=920292
>
> and
>
> Â http://svn.apache.org/viewvc?view=revision&revision=876414
>
> look to be the first two, but so far all they do is move the problem around :-)
>
> I'll dig some more tomorrow.
>
> Bert - CCed you specifically since you've done a lot of work with the
> svn_[dirent|uri]_* APIs as well as the "c:hi"-type fixes, so if you
> think I'm on a fool's errand here (i.e. going to require 1.7 APIs to
> fix) please let me know!
Appears it was simpler than I first thought: The only change needed to
fix basic_tests.py 37 over ra_local on Windows is
http://svn.apache.org/viewvc?view=revision&revision=877014
Running the [ra_local x fsfs] tests in full right now. Assuming there
are no unexpected problems I'll nominate r877014 for backport when
they are finished.
Paul
Received on 2010-04-02 17:46:49 CEST