php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #62925 some issues while --enable-intl
Submitted: 2012-08-24 15:15 UTC Modified: 2012-08-26 20:08 UTC
From: laruence@php.net Assigned: cataphract (profile)
Status: Closed Package: I18N and L10N related
PHP Version: Irrelevant OS:
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: laruence@php.net
New email:
PHP Version: OS:

 

 [2012-08-24 15:15 UTC] laruence@php.net
Description:
------------
1. php-trunk branch build failed:
 source/trunk/ext/intl/dateformat/dateformat_format_object.cpp -o 
ext/intl/dateformat/dateformat_format_object.o
/home/huixinchen/opensource/trunk/ext/intl/dateformat/dateformat_format_object.c
pp:42: error: ‘kFullRelative’ is not a member of ‘icu_3_6::DateFormat’
/home/huixinchen/opensource/trunk/ext/intl/dateformat/dateformat_format_object.c
pp:43: error: ‘kLongRelative’ is not a member of ‘icu_3_6::DateFormat’
/home/huixinchen/opensource/trunk/ext/intl/dateformat/dateformat_format_object.c
pp:44: error: ‘kMediumRelative’ is not a member of ‘icu_3_6::DateFormat’

2. coding standard:
I am not sure, whether the c99 comment is okey, and no footer in source codes

3. test failed , in 5.4-zts, there are some tests failed

=====================================================================
FAILED TEST SUMMARY
---------------------------------------------------------------------
Bug #62082: Memory corruption in internal get_icu_disp_value_src_php() 
[ext/intl/tests/bug62082.phpt]
Cloning datefmt icu <= 4.2 [ext/intl/tests/dateformat_clone.phpt]
datefmt_get_pattern_code and datefmt_set_pattern_code() icu <= 4.2 
[ext/intl/tests/dateformat_get_set_pattern.phpt]
datefmt_set_timezone_id_code() icu <= 4.2 
[ext/intl/tests/dateformat_set_timezone_id.phpt]
numfmt_format() icu <= 4.2 [ext/intl/tests/formatter_format.phpt]
numfmt_format_currency() icu <= 4.2 
[ext/intl/tests/formatter_format_currency.phpt]
numfmt_get/set_attribute() [ext/intl/tests/formatter_get_set_attribute.phpt]
grapheme() [ext/intl/tests/grapheme.phpt]
=====================================================================

Test script:
---------------
none

Expected result:
----------------
none

Actual result:
--------------
none

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2012-08-24 15:15 UTC] laruence@php.net
-Assigned To: +Assigned To: cataphract
 [2012-08-24 15:15 UTC] laruence@php.net
Cataphract, please look at this. thanks
 [2012-08-24 15:19 UTC] laruence@php.net
btw:
$ icu-config --version
3.6
 [2012-08-24 16:54 UTC] cataphract@php.net
php-master raises the minimum ICU version to 4.0 (see UPGRADING).

It's possible that ICU 4.0 has some test failures (it's difficult to write stable tests because the underlying data changes frequently), but it should at least compile.
 [2012-08-25 01:34 UTC] laruence@php.net
Then I think it should be denied in configure ?
 [2012-08-26 20:08 UTC] cataphract@php.net
-Status: Assigned +Status: Closed
 [2012-08-26 20:08 UTC] cataphract@php.net
This bug has been fixed in SVN.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.

Changed configure in 8831b00
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu Oct 31 23:01:28 2024 UTC