Stas Bekman wrote:
Philippe M. Chiasson wrote:
Stas Bekman wrote:
Philippe M. Chiasson wrote: [...]
Moreover now that I think about it I'm not quite sure why in the world someone will want to turn this feature on. Can you please remind me why was it added at all? If someone wants that badly then can just wrap the whole section in the eval {} block.
As far as I can tell/remember, this is simply a mp1 feature that I wanted to
preserve when porting the <Perl> code to mp2.
I agree that it doesn't add much functionnality that an eval {} block wouldn't
allow (except maybe if the code _doesn't_ compile at all.
How about this?
- Implement PerlSectionsNonFatal = 1
- Deprecate StrictPerlSections with a warning, unless MP_COMPAT_1X is selected ?
If we agree that it has no real added value then we don't want it at all.
Unless somebody can come up with a good reason why we need it, I am quite
happy getting rid of this 'feature'
I guess you could check the list archives to see why was it added in first place? Doug is known not to add useless features. Quite possible that we miss the reason.
How about for now we keep things the way they stand when MP_COMPAT_1X=1, and just
deprecate it all, making <Perl> sections strict _always_ and if people want errors
not to abort startup, just use eval {} ?
Yes, but, MP_COMPAT_1X=1 is the default, so your change accomplishes nothing for this default value. We still remain with non-strict sections by default, no?
Also, these two: $Apache::Server::SaveConfig, $Apache::Server::StrictPerlSections need to move to the package they belong to. Which is Apache::PerlSections::. And you can now call it:
Apache::PerlSections::Strict. But that's a separate issue.
-- __________________________________________________________________ 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]
