Bug #77129 for mod_perl: PL_uid breakage on 5.16.0

Fri May 11 07:31:47 2012 zefram [...] fysh.org - Ticket created

Subject:	PL_uid breakage on 5.16.0
Date:	Fri, 11 May 2012 12:31:35 +0100
To:	bug-mod_perl [...] rt.cpan.org
From:	Zefram <zefram [...] fysh.org>

mod_perl won't build on 5.16.0-RC0 because PL_uid and friends have disappeared. (Actually it fails first because of a small bug in Module::CoreList, which I presume will be fixed in RC1; for the purposes of this bug report I'm assuming you've worked around that.) The culprit is four lines in src/modules/perl/modperl_perl.c that assign to these variables on non-WIN32 platforms. The variables are now redundant; mod_perl should simply not attempt the assignments on 5.16.0 or later. Likewise PL_ppid, to which there's a compile-time-conditional assignment immediately following. -zefram

Mon May 21 06:19:46 2012 zefram [...] fysh.org - Correspondence added

Subject:	Re: [rt.cpan.org #77129] AutoReply: PL_uid breakage on 5.16.0
Date:	Mon, 21 May 2012 11:19:34 +0100
To:	Bugs in mod_perl via RT <bug-mod_perl [...] rt.cpan.org>
From:	Zefram <zefram [...] fysh.org>

Patch that I'm using is attached. -zefram

Message body is not shown because sender requested not to inline it.

Mon May 21 09:18:45 2012 MARKF [...] cpan.org - Correspondence added

This patch WFM on the 5.16.0 full release. (though t/api/err_headers_out.t fails, but I believe this to be unrelated, as we saw failures on this with 5.12 too)

Mon May 21 09:18:46 2012 The RT System itself - Status changed from 'new' to 'open'

Wed May 30 08:27:50 2012 shay [...] cpan.org - Correspondence added

Thanks, applied to trunk as rev 1344224.

Tue Jun 05 15:27:23 2012 gozer [...] cpan.org - Correspondence added

Steve Hay fixed this

Tue Jun 05 15:27:25 2012 gozer [...] cpan.org - Status changed from 'open' to 'resolved'