Bug #76535 [Com]: Opcache does not replay compile-time warnings
Edit report at https://bugs.php.net/bug.php?id=76535&edit=1
ID: 76535
Comment by: hrvoje dot novosel at gmail dot com
Reported by: nikic@php.net
Summary: Opcache does not replay compile-time warnings
Status: Open
Type: Bug
Package: opcache
PHP Version: Irrelevant
Block user comment: N
Private report: N
New Comment:
When testing our project on PHP 7.4 I noticed the curly braces deprecation appears only on 1st page
load. I'm guessing this is the same thing as this bug report... are there plans to fix this or
will it stay as is?
Previous Comments:
------------------------------------------------------------------------
[2018-08-20 12:04:55] nikic@php.net
Related To: Bug #76753
------------------------------------------------------------------------
[2018-08-17 10:00:26] cmb@php.net
Related To: Bug #76753
------------------------------------------------------------------------
[2018-06-28 09:39:47] nikic@php.net
If there is a performance concern, I'd be fine with having this behind an ini flag like
opcache.replay_warnings=1.
------------------------------------------------------------------------
[2018-06-28 09:36:45] laruence@php.net
I mean that if there is warning, people(php users) should fix it in there own codes...
anyway, I understand your point, maybe just disable opcache while test warnings?
I was concern that we need pay some costs, like performance reduce to achieve this... like
save_cwd...
if it cost little, then I have no problems....
------------------------------------------------------------------------
[2018-06-28 08:49:54] nikic@php.net
@laruence: My main use for this is testing. To test file cache, you need to run something like
sapi/cli/php run-tests.php -P -c php.ini -d opcache.file_cache=/tmp Zend/
sapi/cli/php run-tests.php -P -c php.ini -d opcache.file_cache=/tmp -d
opcache.validate_timestamps=0 Zend/
However, this causes a lot of test failures even if every thing works "correctly", due to
differences in warnings.
Unless by killing warnings you mean that we have plans to remove them from PHP entirely, I think it
makes sense to implement this.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=76535
--
Edit this bug report at https://bugs.php.net/bug.php?id=76535&edit=1
Thread (8 messages)