# Win snapshot is not built (PHP 5.2) - Thread: https://externals.io/message/23228 - Original archive: https://news-web.php.net/php.internals/23228 - Started by Marian Kostadinov on 2006-05-10T15:46:43+00:00 - Messages: 8 - Reply by email to internals@lists.php.net - More for agents: search with https://externals.io/search.md?q={query} · JSON API at https://externals.io/api · documentation at https://externals.io/llms.txt --- ## #23228 — Marian Kostadinov — 2006-05-10T15:46:43+00:00 URL: https://externals.io/message/23228#23228 Hi all, I wanted to get a 5.2 snapshot for windows but noticed the it did not build. The error in the log is: ext\com_dotnet\com_extension.c(195) : error C2198: 'zend_exception_get_default' : too few actual parameters Can anyone fix it so I can test the fixed __toString behaviour? --- ## #23229 — Pierre — 2006-05-10T15:53:35+00:00 URL: https://externals.io/message/23228#23229 · In reply to: #23228 On Wed, 10 May 2006 18:46:43 +0300 manchokapitancho@gmail.com ("Marian Kostadinov") wrote: > Hi all, > I wanted to get a 5.2 snapshot for windows but noticed the it did not > build. The error in the log is: > > ext\com_dotnet\com_extension.c(195) : error C2198: > 'zend_exception_get_default' : too few actual parameters > > Can anyone fix it so I can test the fixed __toString behaviour? Rob fixed this error earlier today, the next snapshots should work. -- Pierre --- ## #23238 — Marian Kostadinov — 2006-05-11T06:28:59+00:00 URL: https://externals.io/message/23228#23238 · In reply to: #23229 Thanks for the information. Finally I downloaded a new version and __toString is working as expected. There is an issue about error_reporting setting in php.ini. I took my php 5.1 php.ini file and used it for 5.2. Strangely it seemed that `E_STRICT` suddenly turned on. Is this an intended change or a bug? 2006/5/10, Pierre : > > On Wed, 10 May 2006 18:46:43 +0300 > manchokapitancho@gmail.com ("Marian Kostadinov") wrote: > > > […] > > Rob fixed this error earlier today, the next snapshots should work. > > -- Pierre --- ## #23241 — Hannes Magnusson — 2006-05-11T09:13:11+00:00 URL: https://externals.io/message/23228#23241 · In reply to: #23238 > There is an issue about error_reporting setting in php.ini. I took my php > 5.1 php.ini file and used it for 5.2. Strangely it seemed that `E_STRICT` > suddenly turned on. > Is this an intended change or a bug? > `E_STRICT` and `E_RECOVERABLE_ERROR` are now part of `E_ALL` -Hannes --- ## #23259 — Bastian Grupe — 2006-05-11T18:29:56+00:00 URL: https://externals.io/message/23228#23259 · In reply to: #23238 Marian Kostadinov wrote: > There is an issue about error_reporting setting in php.ini. I took my php > 5.1 php.ini file and used it for 5.2. Strangely it seemed that `E_STRICT` > suddenly turned on. > Is this an intended change or a bug? AFAIK it was intended to change `E_ALL` to include `E_STRICT` in 5.2+ So you might like to change error reporting to something like `E_ALL` & ~E_STRICT in order to restore the old behavior. -- Bastian --- ## #23304 — Ilia Alshanetsky — 2006-05-12T14:17:23+00:00 URL: https://externals.io/message/23228#23304 · In reply to: #23259 > Marian Kostadinov wrote: > > […] > > AFAIK it was intended to change `E_ALL` to include `E_STRICT` in 5.2+ I think this was planned for PHP 6 not PHP 5.2. Although I don't see a problem with making this change in 5.2 as well, does anyone have any objections? > Ilia Alshanetsky Advanced Internet Designs Inc. ilia@prohost.org --- ## #23306 — Pierre — 2006-05-12T14:25:42+00:00 URL: https://externals.io/message/23228#23306 · In reply to: #23304 On Fri, 12 May 2006 10:17:23 -0400 ilia@prohost.org (Ilia Alshanetsky) wrote: > > > > > […] > > > I think this was planned for PHP 6 not PHP 5.2. Although I don't see > a problem with making this change in 5.2 as well, does anyone have > any objections? Yes, I do not think it is wise to do it. Something quiet using `E_ALL` in 5.x must remain quiet using 5.2. However, there is maybe some edge cases where it makes sense. I do not have any in mind. -- Pierre --- ## #23323 — Marcus Boerger — 2006-05-12T21:56:13+00:00 URL: https://externals.io/message/23228#23323 · In reply to: #23306 Hello Pierre, hello Ilia, adding `E_STRICT` was an oversight from me. However we must have `E_STRICT` to be able to inform our user base about changes removals in upcoming version. That was decided upon in the PDM iirc. best regards marcus p.s.: this is a new version with some additions and changes as has always been with minor version in php. If we hadn't changes we would call it 5.1.5. Friday, May 12, 2006, 4:25:42 PM, you wrote: > On Fri, 12 May 2006 10:17:23 -0400 > ilia@prohost.org (Ilia Alshanetsky) wrote: > > […] > Yes, I do not think it is wise to do it. Something quiet using `E_ALL` in > 5.x must remain quiet using 5.2. > However, there is maybe some edge cases where it makes sense. I do not > have any in mind. Best regards, Marcus