Stas Bekman wrote:
Geoffrey Young wrote:
and of course need to fix that in mp2.
I'll tacked the mp2 bits this week.
just looking at it now...
it seems that mp2 doesn't respect (Perl)SetEnv PERL5LIB - I can't find it in the code and can't get it to work in the tests.
It's not mp2, it's perl. See the last para:
I'm aware of what PERL5LIB is, thanks :)
comment out PerlSwitches -T and it'll appear.
it doesn't.
and it doesn't because the issue I'm raising is a mod_perl one, and not a perl one. see below.
> as I looked at implementing it, I can't decide on where to put it, since > I never really liked how it behaved in mp1 (as a pseudo-global that > really couldn't be set on a per-<Location> basis even though it's syntax > allowed for it). > > so, my thought now is to abandon SetEnv PERL5LIB and rely on > PerlSwitches -I exclusively for adding @INC paths to the interpreter via > httpd.conf. > > sound ok?
mp1 has a special code to circumvent perl's ignorance of PERL5LIB under -T. We could copy it over, but I'd suggest to run it only at the server startup (only if we detect that -T is On) and never touch it again. We probably need to support that, so that it won't give users a reason not to use -T.
I don't see that in core. I see some special handling for tainting in the test suite, though. but you could be right.
nevertheless, the issue I'm talking about is that using (Perl)SetEnv only populates the subprocess_env table, not %ENV, and if %ENV ever does get populated with the contents of subprocess_env (and it may not under some circumstances) it's too late to have PERL5LIB recognized by the perl interpreter, since it's already running. this is why mp1 manually adds the PERL5LIB from r->subprocess_env to @INC.
the problem with mp1, though, is that
SetEnv PERL5LIB /foo
can occur on a per-<Location> basis. the result is that people _think_ they are setting PERL5LIB for a given uri but they aren't, and it fails silently.
--Geoff
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
