ID:               44219
 Updated by:       [EMAIL PROTECTED]
 Reported By:      kirkland at ubuntu dot com
-Status:           Open
+Status:           Feedback
 Bug Type:         Date/time related
 Operating System: Linux 64 bit
 PHP Version:      5.2.5
 New Comment:

Please try using this CVS snapshot:

  http://snaps.php.net/php5.2-latest.tar.gz
 
For Windows (zip):
 
  http://snaps.php.net/win32/php5.2-win32-latest.zip

For Windows (installer):

  http://snaps.php.net/win32/php5.2-win32-installer-latest.msi




Previous Comments:
------------------------------------------------------------------------

[2008-02-22 20:31:34] kirkland at ubuntu dot com

Test script:
# cat /tmp/foo.php 
<?php
var_dump(strtotime('0000-00-00 00:00:00'));
?>

Built without patch:
# php /tmp/foo.php 
bool(false)


Built with patch:
# ./cli-build/sapi/cli/php /tmp/foo.php 
int(-62169962400)

------------------------------------------------------------------------

[2008-02-22 20:21:41] kirkland at ubuntu dot com

In other words, adding that 3-line patch creates the following test
regressions.  The following tests now fail on 64-bit Linux:

+ Bug #41523 (strtotime('0000-00-00 00:00:00') is parsed as 1999-11-30)
[ext/date/tests/bug41523.phpt]

+ mktime() [3] [ext/date/tests/mktime-3.phpt]

+ strtotime() and mysql timestamps
[ext/date/tests/strtotime-mysql.phpt]

+ strtotime() function [ext/date/tests/strtotime3.phpt]

+ wddx deserialization test [ext/wddx/tests/001.phpt]

------------------------------------------------------------------------

[2008-02-22 19:12:18] kirkland at ubuntu dot com

The diff shown in that pastebin is from two subsequent runs, both on
64-bit Linux.

The only difference between the two is:

+++ php-src/ext/date/lib/timelib.h      Fri Feb 22 09:48:18 2008
...
+#if HAVE_LIMITS_H
+#include <limits.h>
+#endif
...

------------------------------------------------------------------------

[2008-02-22 18:38:26] [EMAIL PROTECTED]

Not really... it's just that results are sometimes different on 32 vs
64 bit. 

------------------------------------------------------------------------

[2008-02-22 18:28:40] kirkland at ubuntu dot com

Description:
------------
I just rolled the patch associated with the fix from Bug #44209:
http://marc.info/?l=php-cvs&m=120367371119481&w=2

However, I see test regressions on 64-bit Linux.  See the differences
here:
http://pastebin.com/pastebin.php?diff=d365cb1ad

Most of the failed tests are in the date/time realm, so I'd think there
might be some concern.  Thoughts?



------------------------------------------------------------------------


-- 
Edit this bug report at http://bugs.php.net/?id=44219&edit=1

Reply via email to