Re: mp2, apache 2.0.46, perl 5.8.0, & gcc 3.3 core dumps on solaris8

2003-06-18 Thread Stas Bekman
[EMAIL PROTECTED] wrote: -8<-- Start Bug Report 8<-- 1. Problem Description: gcc 3.3, solaris 8, perl 5.8.0, apache 2.0.46, mod_perl 1.99_10 make test failed I used gcc 2.95.2 built perl5.8.0 and mod_perl and it works ok. however, I upgraded gcc 3.3 a

Re: [mp2 Patch] ModPerl::Util::undef(coderef)

2003-06-18 Thread Stas Bekman
Philippe M. Chiasson wrote: Following the discussion on Apache::Reload and Apache::Symbol, I've taken out Apache::Symbol::undef from mp1, massaged it a bit and made it into ModPerl::Util::undef for mod_perl 2. Works wonders when you want to undefine subroutines, constant or not, with or without pro

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Stas Bekman
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

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Stas Bekman
I guess Apache::Symbol might need to be re-created for mp2. According to : $> fgrep Apache::Symbpl mp2/docs/deprecated_features.txt + Apache::Symbol: unknown There are only 3 functions in there, and they are: undef sv_name cv_const_sv I don't see why we can't find some room for those somewhere

mp2, apache 2.0.46, perl 5.8.0, & gcc 3.3 core dumps on solaris 8

2003-06-18 Thread twh
-8<-- Start Bug Report 8<-- 1. Problem Description: gcc 3.3, solaris 8, perl 5.8.0, apache 2.0.46, mod_perl 1.99_10 make test failed I used gcc 2.95.2 built perl5.8.0 and mod_perl and it works ok. however, I upgraded gcc 3.3 and rebuilt perl, apache

Patch for FakeRequest

2003-06-18 Thread David Wheeler
Hi All, I use Apache::FakeRequest for testing, of course, but when I have a class that expects an argument to be isa Apache, it of course won't work. This patch fixes it. Could it make it in before mod_perl 1.28? FWIW, I'd be happy to also provide a patch that implements import() so that users

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Stas Bekman
Philippe M. Chiasson wrote: Here is some code I've ripped from an in-house Apache::Reload equivalent... It's a bit convoluted, but does work quite nicely. [...] my $stab = Devel::Symdump->$create($package); Just make sure that you specify Devel::Symdump as a requirement. This code should certainl

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Stas Bekman
What we can do is to put it into 1.28's release and phase out its standalone package-existence. Those wanting the lastest Apache::Reload against the older packages will get it from 1.28. Sounds like a good idea. Hopefully Philippe won't mind to include it in the 1.28 release. I sure do not

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Stas Bekman
Steve Hay wrote: Hi Stas, Stas Bekman wrote: Can you adjust the original patch to use Symbol and not Apache::Symbol, so we can re-use the same code in mp1 and mp2? mp2 doesn't carry Apache::Symbol. No. (Well, not straight-forwardly, anyway.) Apache::Symbol is not a simple Apache-savvy r

[mp2 Patch] ModPerl::Util::undef(coderef)

2003-06-18 Thread Philippe M. Chiasson
Following the discussion on Apache::Reload and Apache::Symbol, I've taken out Apache::Symbol::undef from mp1, massaged it a bit and made it into ModPerl::Util::undef for mod_perl 2. Works wonders when you want to undefine subroutines, constant or not, with or without prototypes. Very usefull for A

Re: cvs commit: modperl-2.0/t/hooks/TestHooks authen.pm authz.pm

2003-06-18 Thread Stas Bekman
Geoffrey Young wrote: -use Apache::Const -compile => qw(OK AUTH_REQUIRED SERVER_ERROR); +use Apache::Const -compile => qw(OK HTTP_UNAUTHORIZED SERVER_ERROR); if you want to go this route, you might as well use HTTP_INTERNAL_SERVER_ERROR as well :) I thought you had a commit access? ;)

Re: [mp2 Patch] Simple changes to current Apache::Reload for mp1& mp2

2003-06-18 Thread Stas Bekman
Philippe M. Chiasson wrote: On Thu, 2003-06-19 at 12:22, Stas Bekman wrote: Philippe M. Chiasson wrote: I needed this simple patch to get Apache::Reload working in mp1. Any objections to at least applying this so it is ready for possible inclusion in mp1's upcoming release? Of course, if Steve H

[email protected]

2003-06-18 Thread Philippe M. Chiasson
On Thu, 2003-06-19 at 12:22, Stas Bekman wrote: > Philippe M. Chiasson wrote: > > I needed this simple patch to get Apache::Reload working in mp1. > > > > Any objections to at least applying this so it is ready for possible > > inclusion in mp1's upcoming release? > > > > Of course, if Steve Hay'

Re: [mp2 Patch] Simple changes to current Apache::Reload for mp1& mp2

2003-06-18 Thread Stas Bekman
Philippe M. Chiasson wrote: I needed this simple patch to get Apache::Reload working in mp1. Any objections to at least applying this so it is ready for possible inclusion in mp1's upcoming release? Of course, if Steve Hay's suggestions make it through, keeping the backwards compatibility would be

Re: cvs commit: modperl-2.0/t/hooks/TestHooks authen.pm authz.pm

2003-06-18 Thread Geoffrey Young
-use Apache::Const -compile => qw(OK AUTH_REQUIRED SERVER_ERROR); +use Apache::Const -compile => qw(OK HTTP_UNAUTHORIZED SERVER_ERROR); if you want to go this route, you might as well use HTTP_INTERNAL_SERVER_ERROR as well :) --Geoff -

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Steve Hay
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

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Philippe M. Chiasson
On Wed, 2003-06-18 at 17:19, Steve Hay wrote: > Hi Philippe, > > Philippe M. Chiasson wrote: > > >Here is some code I've ripped from an in-house Apache::Reload > >equivalent... It's a bit convoluted, but does work quite nicely. > > > Thanks for the code -- it looks excellent to me. no problem >

Re: [Fwd: Apache::Reload Module]

2003-06-18 Thread Steve Hay
Hi Philippe, Philippe M. Chiasson wrote: Here is some code I've ripped from an in-house Apache::Reload equivalent... It's a bit convoluted, but does work quite nicely. Thanks for the code -- it looks excellent to me. I've attached a patch for Apache::Reload that makes use of your code to achiev