Perrin Harkins wrote:
On Wed, 2005-01-05 at 11:07 -0500, Stas Bekman wrote:
that's misleading. The platform is supported. I see nothing wrong with
having different error messages.
I was just thinking of making it easier to test, but you're right, it's
not as clear a message.
How about just:
croak "Unsupported platform: $^O";
Because it's a croak, I think it needs to say "Apache::SizeLimit" in
there.
You are correct. I haven't thought of that. May be just leave die then?
I'll make the changes to use croak and just leave it at that. I'll also
look at moving this from the startup into an official test, so that you
would get a "skipped" on unsupported platforms. It would still just
test loading the module, but it would be less confusing I think.
sure
--
__________________________________________________________________
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]