Hi Jim,

2005/11/16, Jim Gallacher <[EMAIL PROTECTED]>:
> +1 with patch
> Linux gentoo 2.6.12-gentoo-r6
> apache 2.0.54 (mpm-prefork)
> python 2.4.2
> gcc 3.3.6
>
> There are 2 issues with the unit tests in Gentoo that are fixed by the
> attached patch. (Just to be clear, I mean the problems are with the unit
> test code, not with mod_python).
>
> First, test_global_lock uses ab in the test. In Gentoo, ab is named ab2,
> so this test fails. The attached patch just skips the test if ab doesn't
> exist. We can improve on this later.

My bad, I'm the one who put back this test... It was commented out
before mentioning a "bug in ab" with AFAIK the test ran perfectly. I
didn't think about ab being renamed ab2...

> The second issue is with test_req_headers_out, first reported by Dominic
> Wong a couple of weeks ago. The fix is simple, but I want to understand
> why it was failing under Gentoo and not other platforms.
>
> The culprit seems to be the use of DirectoryIndex directive in the
> test.conf. The existence of "DirectoryIndex /tests.py" causes apache to
> segfault. Removing DirectoryIndex and giving the full url in the
> putrequest allows the test to complete successfully.
>
> So my questions are:
>
> 1. Why would DirectoryIndex cause a segfault on gentoo but not other
> platforms?

I don't know why, but isn't it strange to put a slash in front of
tests.py ? Shouldn't the directive be just "DirectoryIndex tests.py" N

> 2. This test is the only one that uses DirectoryIndex. Is there any
> special reason for this?
>
> 3. This test is the only one that uses AddHandler instead of SetHandler.
> Is there a reason for this?
>
> 4. This test is the only one that sets a PythonAccessHandler directive
> in test.conf. Is there a reason for this?
>
> Can anyone offer any insights?

It also uses httplib and not vhost_get. It's as if this test was one
of the first that have been written, and that ways to write better
tests have been improved since (using vhost_get etc.).

Regards,
Nicolas

Reply via email to