I'm not totally convinced by this argument. CGI is definitively a different SAPI than CLI and can behave differently. It was a problem before when we had only one command for both CLI and CGI, but that's not the case anymore.
We should better document CLI and recommend to always install it shell usage is planed (and distros should install it by default as well). On Sat, Sep 5, 2009 at 1:47 PM, Jani Taskinen<jani.taski...@sci.fi> wrote: > It's very legit to use with CGI since you might not be able to run PHP any > other way. So this is definately a bug, CGI should handle it the same as > CLI. > > The bug report clearly explains it too: > > http://bugs.php.net/bug.php?id=49182 > > --Jani > > Andi Gutmans wrote: >> >> Shebang is for command line scripts (php-cli). It does not make sense to >> support it for Web server scripts. It just adds unnecessary >> code/complexity to that code base. Removing the support from php-cgi was >> really a remnant of the old days when cli and cgi were the same SAPI. >> I think we are better off this way. >> >> Andi >> >>> -----Original Message----- >>> From: Joey Smith [mailto:j...@joeysmith.com] >>> Sent: Friday, September 04, 2009 1:35 AM >>> To: internals@lists.php.net >>> Subject: Re: [PHP-DEV] shebang skipping in 5.3.0 >>> >>> I definitely had the wrong changeset - sorry, Nuno. :) Looks >>> like maybe 273177 is the problem child. >>> >>> http://tinyurl.com/lewcft >>> >>> >>> On Fri, Sep 04, 2009 at 09:25:52AM +0100, Scott MacVicar wrote: >>>> >>>> On 4 Sep 2009, at 09:16, Joey Smith <j...@joeysmith.com> wrote: >>>> >>>>> I can understand not having the 'shebang skipping' code >>>>> in both the SAPI *and* the scanner, but we probably >>>>> need to have it in at least ONE of them. :) >>>>> >>>>> Per his email[1] almost a year ago, Dmitry removed the >>>>> shebang line check from sapi/cgi/cgi_main.c in changeset >>>>> 264153, saying: >>>>> Removed shebang line check from CGI sapi (it is >>>>> checked by scanner) >>>>> >>>>> In http://tinyurl.com/me8528 (changeset 262275), nlopess >>>>> removed the code from the scanner. >>>>> >>>>> Oops? >>>> >>>> What's the problem your having? The skip code is still there just in >> >> a >>>> >>>> different bit. >>>> >>>> Scott >>>> >>>> -- >>>> PHP Internals - PHP Runtime Development Mailing List >>>> To unsubscribe, visit: http://www.php.net/unsub.php >>>> >>> -- >>> PHP Internals - PHP Runtime Development Mailing List >>> To unsubscribe, visit: http://www.php.net/unsub.php >> >> > > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > > -- Pierre http://blog.thepimp.net | http://www.libgd.org -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
