/[Apache-SVN]
ViewVC logotype

Revision 1717474


Jump to revision: Previous Next
Author: stevehay
Date: Tue Dec 1 17:39:43 2015 UTC (8 years, 5 months ago)
Changed paths: 6
Log Message:
Add support for Perl 5.22.x.

As outlined by Leon Timmermans in [perl #123687]:

  lookup %ENV's magic, and then replace the pointer to PL_vtbl_env with
  a pointer to MP_vtbl_env. You may have to add some svt_copy magic to
  make it cast MP_vtbl_envelem instead of PL_vtbl_envelem on the elements.

with an added svt_local for the 'local %ENV' tests.

While at it, augment t/modperl/env.t to check that deleting %ENV elements really removes them from subprocess_env. This highlights the need for modifying their vtable, currently in the modperl_envelem_tie() macro.

(MP_vtbl_envelem probably shouldn't be a global variable, but the modperl_envelem_tie() macro needs it, and the perl vtables are global too anyway. I've made MP_vtbl_env global too for symmetry.)

Based on ++Niko Tyni's 0001-Steps-at-Perl-5.22-compatibility-take-3.patch in [rt.cpan.org #101962].

Bug: https://rt.cpan.org/Public/Bug/Display.html?id=101962
Bug: https://rt.perl.org/Ticket/Display.html?id=123687


Changed paths

Path Details
Directoryperl/modperl/trunk/Changes modified , text changed
Directoryperl/modperl/trunk/src/modules/perl/mod_perl.c modified , text changed
Directoryperl/modperl/trunk/src/modules/perl/modperl_env.c modified , text changed
Directoryperl/modperl/trunk/src/modules/perl/modperl_env.h modified , text changed
Directoryperl/modperl/trunk/src/modules/perl/modperl_perl.c modified , text changed
Directoryperl/modperl/trunk/t/response/TestModperl/env.pm modified , text changed

infrastructure at apache.org
ViewVC Help
Powered by ViewVC 1.1.26