perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Steve Hay <steve.m....@googlemail.com>
Subject Re: t/compat/conn_rec.t failure in trunk with Apache 2.4
Date Wed, 13 Aug 2014 13:15:13 GMT
On 13 August 2014 09:23, Steve Hay <steve.m.hay@googlemail.com> wrote:
> On 8 August 2014 07:43, Jan Kalu┼ża <jkaluza@redhat.com> wrote:
>> On 08/07/2014 01:32 PM, Niko Tyni wrote:
>>>
>>> Hi,
>>>
>>> I'm trying to get the current mod_perl trunk to build on Debian unstable
>>> (with Apache 2.4.10 and Perl 5.18.2), and I'm seeing a failure in
>>> t/compat/conn_rec.t. It's getting an internal server error at test 3,
>>> with this in t/logs/error_log:
>>>
>>>   [Thu Aug 07 14:15:39.916269 2014] [perl:error] [pid 19060:tid
>>> 140108267251456] [client 127.0.0.1:36626] overriding
>>> Apache2::Connection::remote_addr didn't return a CODE ref at
>>> /home/niko/tmp/libapache2-mod-perl2/blib/lib/Apache2/compat.pm line 252.\n
>>>
>>> So it's the remote_addr vs. client_addr thing again.
>>>
>>> The test used to work with the httpd24 branch at r1449661, and it
>>> looks like it broke with r1497279, which restored 2.2 compatibility
>>> in lib/Apache2/compat.pm
>>>
>>>   http://svn.apache.org/viewvc?view=revision&revision=1497279
>>>
>>> If I understand this correctly, Apache2::compat is supposed to provide
>>> mod_perl 1.0 backward compatibility, so we want to offer remote_addr()
>>> there even with Apache 2.4. Does the attached patch make sense? It works
>>> for me on 2.4 and I'd expect it to work with 2.2 too (but I haven't
>>> verified that.)
>>
>>
>> Hi,
>>
>> yes it does. I didn't do that because of my poor Perl knowledge, so I had no
>> idea how to detect httpd-2.4 in that particular part of code. I will commit
>> the patch next week (right now I'm on conference and I would not like to
>> commit the patch blindly).
>>
>
> Looks good to me too. I will also test on 2.2 and 2.4...

This works for me with both 2.4.9 and 2.2.27 (using Perl 5.21.0 on
Windows, with VC++ 2010) but I will refrain from applying it since Jan
said he'd test and apply it.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Mime
View raw message