I have written a small patch to solve this problem. Please have a look at it. If it looks good, I will write tests and submit the patch.
The scheme is :
1. Populate %ENV before all <Perl>..</Perl>, PerlRequire and PerlModule. 2. After every <Perl>..</Perl>, PerlRequire and PerlModule, sync server tables ( scfg->SetEnv & scfg->PassEnv ) with %ENV.
Very nice work, Pratik. The only problem is that it syncs SetEnv|PassEvn <=> %ENV completely twice for each <Perl>..</Perl>, PerlRequire and PerlModule, which is not very effective. It would be nice to keep track of just what keys have changed.
Regardless, we need to have the tests, since it looks that what you need is the right thing.
Of course it'd be nice to have a more efficient solution.
also I think you shouldn't use modperl_env_configure_server() since it does more than that (and that extra thing doesn't need to happen so many times since it doesn't change). Instead I think there should be:
modperl_env_sync_env2tables modperl_env_sync_tables2env
or something like that.
-- __________________________________________________________________ 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]
