Stas Bekman wrote:
Philippe M. Chiasson wrote:
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?
Correct, I just realized that MP_COMPAT_1X was on by default, dhu.
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.
I'll change those names sepraratly in just a moment, it does make much more sense.
But in regards to the strict perl sections issue, what are we doing about it?
$Apache::Server::StrictPerlSections
* Get rid of it ?
* Keep it only when MP_COMPAT_1X=1
* Warn about it being deprecated in favor of eval {}$Apache::PerlSections::Strict on by default, and optionnaly disableable ?
-- -------------------------------------------------------------------------------- Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5 http://gozer.ectoplasm.org/ F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5
signature.asc
Description: OpenPGP digital signature
