On Tue, November 14, 2006 4:42 am, Dmitry Stogov wrote: > The following patch adds new INI directive for CGI SAPI. > It may enable/disable check for shebang line (#! /usr/bin/php) and as > result > improve performance of FastCGI. > > I set the default value for this directive to 1 (that means check file > on > each request) to not break previous behavior. > However may be it is better to set default value to 0. That means > break > previous behavior but improve performance by default.
For sure, default to non-BC-breaking setting of 1. That shouldn't even be a question. :-) Is another .ini setting worth 4% increase in performance? Not to me, but I'm not the guy that needs 4% CGI improvement on any servers I run :-) But another .ini setting doesn't bother me as it does some, so I'm fine with that. Killing shebang entirely in 6.0 --- I dunno... Seems like there's probably a lot of cron jobs out there relying on it, that nobody will think of when you ask them, but as soon as you pull the plug, you're gonna get a zillion complaints... And as far as not including it in the php.ini, that seems to me like it would cause more problems than solutions. It ends up seeming to php.ini readers like an "Undocumented Feature" or something, and you just generate more questions than adding one more setting to php.ini -- Some people have a "gift" link here. Know what I want? I want you to buy a CD from some starving artist. http://cdbaby.com/browse/from/lynch Yeah, I get a buck. So? -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
