if somebody wonders, I just forgot to fix the commit log, that should be: ignore the tests for > 127, not >= 127
On Thu, Oct 20, 2011 at 8:09 PM, Ferenc Kovacs <tyr...@php.net> wrote: > tyrael Thu, 20 Oct 2011 18:09:44 +0000 > > Revision: http://svn.php.net/viewvc?view=revision&revision=318268 > > Log: > there is a length limit on the shebang line, ignore the tests for >= 127 > > Changed paths: > U php/php-src/branches/PHP_5_3/sapi/cli/tests/021.phpt > U php/php-src/branches/PHP_5_4/sapi/cli/tests/021.phpt > U php/php-src/trunk/sapi/cli/tests/021.phpt > > Modified: php/php-src/branches/PHP_5_3/sapi/cli/tests/021.phpt > =================================================================== > --- php/php-src/branches/PHP_5_3/sapi/cli/tests/021.phpt 2011-10-20 > 16:20:42 UTC (rev 318267) > +++ php/php-src/branches/PHP_5_3/sapi/cli/tests/021.phpt 2011-10-20 > 18:09:44 UTC (rev 318268) > @@ -6,6 +6,10 @@ > if (substr(PHP_OS, 0, 3) == 'WIN') { > die ("skip not for Windows"); > } > + > +if (strlen("#!".getenv('TEST_PHP_EXECUTABLE')) > 127) { > + die ("skip shebang is too long, see > http://www.in-ulm.de/~mascheck/various/shebang/#results";); > +} > ?> > --FILE-- > <?php > > Modified: php/php-src/branches/PHP_5_4/sapi/cli/tests/021.phpt > =================================================================== > --- php/php-src/branches/PHP_5_4/sapi/cli/tests/021.phpt 2011-10-20 > 16:20:42 UTC (rev 318267) > +++ php/php-src/branches/PHP_5_4/sapi/cli/tests/021.phpt 2011-10-20 > 18:09:44 UTC (rev 318268) > @@ -6,6 +6,10 @@ > if (substr(PHP_OS, 0, 3) == 'WIN') { > die ("skip not for Windows"); > } > + > +if (strlen("#!".getenv('TEST_PHP_EXECUTABLE')) > 127) { > + die ("skip shebang is too long, see > http://www.in-ulm.de/~mascheck/various/shebang/#results";); > +} > ?> > --FILE-- > <?php > > Modified: php/php-src/trunk/sapi/cli/tests/021.phpt > =================================================================== > --- php/php-src/trunk/sapi/cli/tests/021.phpt 2011-10-20 16:20:42 UTC > (rev 318267) > +++ php/php-src/trunk/sapi/cli/tests/021.phpt 2011-10-20 18:09:44 UTC > (rev 318268) > @@ -6,6 +6,10 @@ > if (substr(PHP_OS, 0, 3) == 'WIN') { > die ("skip not for Windows"); > } > + > +if (strlen("#!".getenv('TEST_PHP_EXECUTABLE')) > 127) { > + die ("skip shebang is too long, see > http://www.in-ulm.de/~mascheck/various/shebang/#results";); > +} > ?> > --FILE-- > <?php > > > -- > PHP CVS Mailing List (http://www.php.net/) > To unsubscribe, visit: http://www.php.net/unsub.php >
