Edit report at https://bugs.php.net/bug.php?id=61610&edit=1

 ID:                 61610
 Comment by:         mattfic...@php.net
 Reported by:        a...@php.net
 Summary:            Test ext\date\tests\date_default_timezone_get-1.diff
                     fails
 Status:             Open
 Type:               Bug
 Package:            Date/time related
 Operating System:   Windows
 PHP Version:        5.3.10
 Block user comment: N
 Private report:     N

 New Comment:

The second patch passes on Windows 7 and Gentoo Linux (x86 & x64) on 
php-5-3-r0f180a6.


Previous Comments:
------------------------------------------------------------------------
[2012-04-10 01:56:57] mattfic...@php.net

There is a slight typo on the first patch(date_default_timezone_get-2 instead 
of date_default_timezone_get-1).

The second patch fixes it

------------------------------------------------------------------------
[2012-04-10 01:55:55] mattfic...@php.net

The following patch has been added/updated:

Patch Name: bug61610-2.diff.patch.txt
Revision:   1334022955
URL:        
https://bugs.php.net/patch-display.php?bug=61610&patch=bug61610-2.diff.patch.txt&revision=1334022955

------------------------------------------------------------------------
[2012-04-03 18:13:55] a...@php.net

Tested on debian and win7 both x86.

------------------------------------------------------------------------
[2012-04-03 18:01:07] a...@php.net

The behaviour on windows is to select an arbitrary timezone from the current 
system settings. This gives no chance to hardcode the timezone name, for 
instance for UTC+1 it could choose from the multiple names like Europe/Berlin 
or Europe/Paris . For this reason the test is parametrized so there is no 
hardcoded timezone data.

The original test made to be skipped on windows and a duplicate was made for 
windows.

------------------------------------------------------------------------
[2012-04-03 17:59:10] a...@php.net

The following patch has been added/updated:

Patch Name: 61610.diff
Revision:   1333475950
URL:        
https://bugs.php.net/patch-display.php?bug=61610&patch=61610.diff&revision=1333475950

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


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=61610


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

Reply via email to