Re: early perl startup in vhost on win32

[Stas Bekman]

Steve Hay wrote:
Stas Bekman wrote:


Steve, Randy, can you please try with this patch. This is against the conf 
file with:

<VirtualHost _default_:8542>
PerlOptions +Parent
<Perl>
  1;
</Perl>
</Virtualhost>
The Good News:  The patch below fixes *both* of the short conf files 
that we've been playing with.  Running "apache.exe -t ..." was 
previously enough to produce the crash; now I can even start the server 
up (i.e. without the "-t" option) without error.
Actually, I wonder how does it fix this case:

<VirtualHost _default_:8542>
        PerlOptions +Parent
</Virtualhost>
PerlLoadModule TestDirective::perlloadmodule
it the one I have sent affects only perl sections. Shouldn't you apply a 
similar patch to the MP_CMD_SRV_DECLARE(modules) part

--- src/modules/perl/modperl_cmd.c      19 Dec 2003 01:17:31 -0000      1.52
+++ src/modules/perl/modperl_cmd.c      21 Jan 2004 09:38:39 -0000
@@ -116,6 +116,7 @@
 #ifdef USE_ITHREADS
         /* XXX: .htaccess support cannot use this perl with threaded MPMs */
         dTHXa(scfg->mip->parent->perl);
+        PERL_SET_CONTEXT(aTHX);
 #endif
         MP_TRACE_d(MP_FUNC, "load PerlModule %s\n", arg);
not sure about the restoring the context part, try to play with it. 
MP_CMD_SRV_DECLARE(modules) and MP_CMD_SRV_DECLARE(perldo) are very similar in 
the way they are invoked and the way they use the context. (in fact these are 
the two places where you have/had the problems).

__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]