Re: release blockers?

On 10/16/2014 10:02 AM, Steve Hay wrote:
> On 16 September 2014 08:08, Jan Kaluža <[email protected]
> <mailto:[email protected]>> wrote:
>
>     On 09/12/2014 07:05 PM, Steve Hay wrote:
>
>         On 12 September 2014 16:15, Niko Tyni <[email protected]
>         <mailto:[email protected]>> wrote:
>
>             On Fri, Sep 12, 2014 at 05:01:36PM +0200, Jan Kaluža wrote:
>
>                 On 08/27/2014 11:55 AM, Jan Kaluža wrote:
>
>
>                     since we have mod_perl which compiles and works with
>                     httpd-2.4, what are
>                     the current blockers before we do some release with
>                     initial httpd-2.4
>                     support?
>
>
>                 Does the silence mean that we have no blockers? :)
>
>
>             Is t/perl/ithreads3.t working for everybody else?
>
>             http://mail-archives.apache.__org/mod_mbox/perl-dev/201408.__mbox/%[email protected]%__3E
>             <http://mail-archives.apache.org/mod_mbox/perl-dev/201408.mbox/%[email protected]%3E>
>
>
>         Works for me last time I tried, but t/perl/ithreads* are not in the
>         release tarballs anyway, I think (because of persistent trouble with
>         them historically).
>
>
> It doesn't actually "work for me", of course: It just doesn't fail for
> me since the test requires the worker MPM, so is skipped on Windows!
>
>
>
>     I've tried it with worker MPM and it fails for me, but I think the
>     test is somehow broken. The comment in ithreads3.pm
>     <http://ithreads3.pm> says:
>
>          # now add an extra PerlCleanupHandler. It is run each time the
>          # interp is released. So it is run after Trans, MapToStorage and
>          # Fixup. In the response phase $counter should be 5. After Response
>          # it is run again but that is after.
>
>     But when checking the code for PerlCleanupHandler, it seems to be
>     run once the request request_rec.pool is destroyed, which is in
>     contradiction with the comment.
>
>     It then continues with:
>
>          # This used to eat up all interpreters because
>          # modperl_interp_unselect
>          # calls modperl_config_request_cleanup that allocates a new interp
>          # to handle the cleanup.
>
>     This is also not a true in current code. modperl_interp_unselect
>     does not call modperl_config_request_cleanup if I understand the
>     current code.
>
>     I would say the test is outdated and I have no clue what was its
>     goal. It clearly tries to count how many times are the handlers
>     executed, but in its current state, it is not useful, because it
>     tests the code paths we no longer have.
>
>
> Now omitted from the distribution as per Niko's suggestion: Revision
> 1632231.

Great, thanks. Do you know anything else we have to do/fix before 
release? :) I would like to so see some release with 2.4.x support soon :).

Regards,
Jan Kaluza