perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Stas Bekman <>
Subject Re: [Fwd: Apache::Reload Module]
Date Thu, 19 Jun 2003 06:18:42 GMT
Steve Hay wrote:
> Hi Philippe,
> Philippe M. Chiasson wrote:
>> On Wed, 2003-06-18 at 17:19, Steve Hay wrote:
>>> I've attached a patch for Apache::Reload that makes use of your code 
>>> to achieve what I'm after.  The patch is against the Apache::Reload 
>>> currently in mp2 cvs.  Things to note:
>>> [snip]
>> Can you instead look at this patch:
>> And integrate that into an mod_perl 1 & 2 Apache::Reload instead? I
>> would rather not have to maintain 2 ever so slightly different versions
>> of Apache::Reload if that can be avoided.
> Done. Attached.
> I had to alter the patch that you cited above very slightly: it defers 
> the import of the OK constant until run-time, which made it choke on the 
> bareword OK (since its running under "use strict").  I've therefore 
> pushed the mp1/mp2 conditional stuff inside a BEGIN {} block.
> I've also added a couple of extra lines to conditionally setup an 
> undef_func() -- under mp1 it is just Apache::Symbol::undef, under mp2 it 
> will be ModPerl::Util::undef which you suggested in the rest of your 
> mail (below).
> The only other thing that I was using from Apache::Symbol was the 
> file2class() function. I've now included that in Apache::Reload itself 
> as module_to_package(), complementing the package_to_module() that it 
> already had.

If this is useful in other places we can add it to ModPerl::Util:: perhaps we 
should make further subclassing of that namespace.

> +use B;
> +use Devel::Symdump;

Hmm, do these available for perl 5.005_03? Have you tested with that perl 
version? mp1 has to support perl 5.005_03 I think. Or was it 5.004?

Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker     mod_perl Guide --->

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message