Re: ra_serf now passes all tests was Re: svn commit: r22796 - trunk/subversion/tests/cmdline

On 12/24/06, Lieven Govaerts <svnlgo@mobsol.be> wrote:
> 1. update_tests.py 29 fails with an assertion in libsvn_ra_serf/commit.c
> line 1216.Now this test is supposed to fail anyway, and as far as I can
> see the assertion is triggered exactly because of the bug that the test
> is reproducing. It's a bit annoying because an assertion holds the test
> run (when you have a debugger installed), but the best way to fix it
> would be to fix the actual bug.

"Fixed" in r28804.

> 2. copy_tests.py.py 25 fails on Windows, because the path in the error
> message is printed with '\\' instead of '/' as expected. Now I can
> easily fix the test to support this, but doesn't svn always print paths
> with forward slashes, even on Windows?

Fixed in r28805. We now won't localize the path if it's a 404; for
the other case right below it (i.e. out-of-date), I think it makes
sense to keep localizing the path as its local. Open to suggestions
though.

> I'll start using ra_serf from now on on Windows.

Lemme know how it fares for you. =)

Thanks. -- justin

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Dec 25 01:56:28 2006