diff options
| -rw-r--r-- | PHPINFO | 4 | ||||
| -rw-r--r-- | REFLECTION | 29 | ||||
| -rw-r--r-- | msgpack-php8.patch | 628 | ||||
| -rw-r--r-- | php-pecl-msgpack.spec | 238 |
4 files changed, 138 insertions, 761 deletions
@@ -3,11 +3,13 @@ msgpack MessagePack Support => enabled Session Support => enabled -extension Version => 2.1.1 +MessagePack APCu Serializer ABI => 0 +extension Version => 3.0.0 header Version => 3.2.0 Directive => Local Value => Master Value msgpack.error_display => On => On msgpack.php_only => On => On +msgpack.assoc => On => On msgpack.illegal_key_insert => Off => Off msgpack.use_str8_serialization => On => On @@ -1,4 +1,4 @@ -Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { +Extension [ <persistent> extension #87 msgpack version 3.0.0 ] { - INI { Entry [ msgpack.error_display <ALL> ] @@ -7,6 +7,9 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { Entry [ msgpack.php_only <ALL> ] Current = '1' } + Entry [ msgpack.assoc <ALL> ] + Current = '1' + } Entry [ msgpack.illegal_key_insert <ALL> ] Current = '0' } @@ -15,8 +18,9 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { } } - - Constants [1] { + - Constants [2] { Constant [ int MESSAGEPACK_OPT_PHPONLY ] { -1001 } + Constant [ int MESSAGEPACK_OPT_ASSOC ] { -1002 } } - Functions { @@ -30,7 +34,7 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { - Parameters [2] { Parameter #0 [ <required> $str ] - Parameter #1 [ <optional> $object ] + Parameter #1 [ <optional> $object = <default> ] } } Function [ <internal:msgpack> function msgpack_pack ] { @@ -43,7 +47,7 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { - Parameters [2] { Parameter #0 [ <required> $str ] - Parameter #1 [ <optional> $object ] + Parameter #1 [ <optional> $object = <default> ] } } } @@ -51,8 +55,9 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { - Classes [2] { Class [ <internal:msgpack> class MessagePack ] { - - Constants [1] { + - Constants [2] { Constant [ public int OPT_PHPONLY ] { -1001 } + Constant [ public int OPT_ASSOC ] { -1002 } } - Static properties [0] { @@ -68,7 +73,7 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { Method [ <internal:msgpack, ctor> public method __construct ] { - Parameters [1] { - Parameter #0 [ <optional> $opt ] + Parameter #0 [ <optional> $opt = <default> ] } } @@ -91,7 +96,7 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { - Parameters [2] { Parameter #0 [ <required> $str ] - Parameter #1 [ <optional> $object ] + Parameter #1 [ <optional> $object = <default> ] } } @@ -121,11 +126,11 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { Method [ <internal:msgpack, ctor> public method __construct ] { - Parameters [1] { - Parameter #0 [ <optional> $opt ] + Parameter #0 [ <optional> $opt = <default> ] } } - Method [ <internal:msgpack, dtor> public method __destruct ] { + Method [ <internal:msgpack> public method __destruct ] { - Parameters [0] { } @@ -149,15 +154,15 @@ Extension [ <persistent> extension #86 msgpack version 2.1.1 ] { Method [ <internal:msgpack> public method execute ] { - Parameters [2] { - Parameter #0 [ <optional> $str ] - Parameter #1 [ <optional> &$offset ] + Parameter #0 [ <optional> $str = <default> ] + Parameter #1 [ <optional> &$offset = <default> ] } } Method [ <internal:msgpack> public method data ] { - Parameters [1] { - Parameter #0 [ <optional> $object ] + Parameter #0 [ <optional> $object = <default> ] } } diff --git a/msgpack-php8.patch b/msgpack-php8.patch deleted file mode 100644 index 864692a..0000000 --- a/msgpack-php8.patch +++ /dev/null @@ -1,628 +0,0 @@ -From e3ef2010a9ffb8bcc46a3d5a5a93ad1b048e3f96 Mon Sep 17 00:00:00 2001 -From: Remi Collet <remi@remirepo.net> -Date: Wed, 15 Apr 2020 16:58:27 +0200 -Subject: [PATCH] fix build with php 8 - ---- - msgpack_convert.c | 20 ++++++++++++++++++++ - 1 file changed, 20 insertions(+) - -diff --git a/msgpack_convert.c b/msgpack_convert.c -index c89aff5..93e55b2 100644 ---- a/msgpack_convert.c -+++ b/msgpack_convert.c -@@ -64,7 +64,15 @@ static inline int msgpack_convert_long_to_properties(HashTable *ht, zval *object - *properties = NULL; - } - ZVAL_LONG(&key_zv, key_index); -+#if PHP_VERSION_ID < 80000 - zend_std_write_property(object, &key_zv, val, NULL); -+#else -+ { -+ zend_string *key = zval_get_string(&key_zv); -+ zend_std_write_property(Z_OBJ_P(object), key, val, NULL); -+ zend_string_release(key); -+ } -+#endif - return SUCCESS; - } - /* }}} */ -@@ -87,7 +95,11 @@ static inline int msgpack_convert_string_to_properties(zval *object, zend_string - zend_update_property_ex(ce, object, key, val); - return_code = SUCCESS; - } else { -+#if PHP_VERSION_ID < 80000 - zend_std_write_property(object, &pub_name, val, NULL); -+#else -+ zend_std_write_property(Z_OBJ_P(object), key, val, NULL); -+#endif - return_code = FAILURE; - } - zend_hash_add(var, Z_STR(pub_name), val); -@@ -358,7 +370,11 @@ int msgpack_convert_object(zval *return_value, zval *tpl, zval *value) /* {{{ */ - } ZEND_HASH_FOREACH_END(); - - /* index */ -+#if PHP_VERSION_ID < 80000 - properties = Z_OBJ_HT_P(return_value)->get_properties(return_value); -+#else -+ properties = Z_OBJ_HT_P(return_value)->get_properties(Z_OBJ_P(return_value)); -+#endif - if (HASH_OF(tpl)) { - properties = HASH_OF(tpl); - } -@@ -433,7 +449,11 @@ int msgpack_convert_object(zval *return_value, zval *tpl, zval *value) /* {{{ */ - HashTable *properties = NULL; - HashPosition prop_pos; - -+#if PHP_VERSION_ID < 80000 - properties = Z_OBJ_HT_P(return_value)->get_properties(return_value); -+#else -+ properties = Z_OBJ_HT_P(return_value)->get_properties(Z_OBJ_P(return_value)); -+#endif - zend_hash_internal_pointer_reset_ex(properties, &prop_pos); - - if (msgpack_convert_long_to_properties(HASH_OF(return_value), return_value, &properties, &prop_pos, 0, value, NULL) != SUCCESS) { -From f933cf5df92dbb2da8665905f1f44317f62a4765 Mon Sep 17 00:00:00 2001 -From: Michael Wallner <mike@php.net> -Date: Fri, 31 Jul 2020 06:51:51 +0200 -Subject: [PATCH] remove interned strings optimization causing exhaustive - memory usage - -fixes #153 and possibly fixes #147 ---- - msgpack_unpack.c | 10 +--------- - 1 file changed, 1 insertion(+), 9 deletions(-) - -diff --git a/msgpack_unpack.c b/msgpack_unpack.c -index c743078..8dc9480 100644 ---- a/msgpack_unpack.c -+++ b/msgpack_unpack.c -@@ -12,9 +12,6 @@ - #if PHP_VERSION_ID < 70400 - # define zval_try_get_string zval_get_string - #endif --#if PHP_VERSION_ID < 70200 --# define zend_string_init_interned zend_string_init --#endif - - typedef struct { - zend_long used_slots; -@@ -523,12 +520,7 @@ int msgpack_unserialize_str(msgpack_unserialize_data *unpack, const char* base, - ZVAL_EMPTY_STRING(*obj); - } else { - /* TODO: check malformed input? */ -- if (len < 1<<8) { -- zend_string *zs = zend_string_init_interned(data, len, 0); -- ZVAL_STR(*obj, zs); -- } else { -- ZVAL_STRINGL(*obj, data, len); -- } -+ ZVAL_STRINGL(*obj, data, len); - } - - return 0; -From 780b0b457693180f5f8a82ef1aef02f23f322e03 Mon Sep 17 00:00:00 2001 -From: Remi Collet <remi@remirepo.net> -Date: Thu, 3 Sep 2020 09:37:26 +0200 -Subject: [PATCH] more fix for PHP 8 - ---- - msgpack_class.c | 2 +- - msgpack_convert.c | 17 +++++++++-------- - msgpack_pack.c | 4 ++-- - msgpack_unpack.c | 8 ++++++-- - php_msgpack.h | 6 ++++++ - 5 files changed, 24 insertions(+), 13 deletions(-) - -diff --git a/msgpack_class.c b/msgpack_class.c -index 69f3803..1b0ffe7 100644 ---- a/msgpack_class.c -+++ b/msgpack_class.c -@@ -274,7 +274,7 @@ static ZEND_METHOD(msgpack, unpacker) /* {{{ */ { - ZVAL_STRING(&func_name, "__construct"); - - object_init_ex(return_value, msgpack_unpacker_ce); -- call_user_function_ex(CG(function_table), return_value, &func_name, &construct_return, 1, args, 0, NULL); -+ call_user_function(CG(function_table), return_value, &func_name, &construct_return, 1, args); - - zval_ptr_dtor(&func_name); - } -diff --git a/msgpack_convert.c b/msgpack_convert.c -index 93e55b2..8f9e1d0 100644 ---- a/msgpack_convert.c -+++ b/msgpack_convert.c -@@ -36,7 +36,7 @@ static inline int msgpack_convert_long_to_properties(HashTable *ht, zval *object - - if (msgpack_convert_array(&tplval, data, dataval) == SUCCESS) { - zend_hash_move_forward_ex(props, prop_pos); -- zend_update_property(Z_OBJCE_P(object), object, prop_name, prop_len, &tplval); -+ zend_update_property(Z_OBJCE_P(object), OBJ_FOR_PROP(object), prop_name, prop_len, &tplval); - return SUCCESS; - } - return FAILURE; -@@ -45,14 +45,14 @@ static inline int msgpack_convert_long_to_properties(HashTable *ht, zval *object - { - if (msgpack_convert_object(&tplval, data, val) == SUCCESS) { - zend_hash_move_forward_ex(props, prop_pos); -- zend_update_property(Z_OBJCE_P(object), object, prop_name, prop_len, &tplval); -+ zend_update_property(Z_OBJCE_P(object), OBJ_FOR_PROP(object), prop_name, prop_len, &tplval); - return SUCCESS; - } - return FAILURE; - } - default: - zend_hash_move_forward_ex(props, prop_pos); -- zend_update_property(Z_OBJCE_P(object), object, prop_name, prop_len, val); -+ zend_update_property(Z_OBJCE_P(object), OBJ_FOR_PROP(object), prop_name, prop_len, val); - return SUCCESS; - } - } -@@ -89,10 +89,10 @@ static inline int msgpack_convert_string_to_properties(zval *object, zend_string - prot_name = zend_mangle_property_name("*", 1, ZSTR_VAL(key), ZSTR_LEN(key), 1); - - if (zend_hash_find(propers, priv_name) != NULL) { -- zend_update_property_ex(ce, object, key, val); -+ zend_update_property_ex(ce, OBJ_FOR_PROP(object), key, val); - return_code = SUCCESS; - } else if (zend_hash_find(propers, prot_name) != NULL) { -- zend_update_property_ex(ce, object, key, val); -+ zend_update_property_ex(ce, OBJ_FOR_PROP(object), key, val); - return_code = SUCCESS; - } else { - #if PHP_VERSION_ID < 80000 -@@ -310,8 +310,9 @@ int msgpack_convert_object(zval *return_value, zval *tpl, zval *value) /* {{{ */ - fci.retval = &retval; - fci.param_count = 0; - fci.params = ¶ms; -+#if PHP_VERSION_ID < 80000 - fci.no_separation = 1; -- -+#endif - #if PHP_VERSION_ID < 70300 - fcc.initialized = 1; - #endif -@@ -434,10 +435,10 @@ int msgpack_convert_object(zval *return_value, zval *tpl, zval *value) /* {{{ */ - return FAILURE; - } - -- zend_update_property_ex(ce, return_value, str_key, &nv); -+ zend_update_property_ex(ce, OBJ_FOR_PROP(return_value), str_key, &nv); - zval_ptr_dtor(&nv); - } else { -- zend_update_property(ce, return_value, prop_name, prop_len, aryval); -+ zend_update_property(ce, OBJ_FOR_PROP(return_value), prop_name, prop_len, aryval); - } - num_key++; - } ZEND_HASH_FOREACH_END(); -diff --git a/msgpack_pack.c b/msgpack_pack.c -index c5da6d9..f9d1ad0 100644 ---- a/msgpack_pack.c -+++ b/msgpack_pack.c -@@ -223,7 +223,7 @@ static inline void msgpack_serialize_array(smart_str *buf, zval *val, HashTable - if (object) { - #if PHP_VERSION_ID >= 70400 - if (Z_OBJ_HANDLER_P(val, get_properties_for)) { -- ht = Z_OBJ_HANDLER_P(val, get_properties_for)(val, ZEND_PROP_PURPOSE_ARRAY_CAST); -+ ht = Z_OBJ_HANDLER_P(val, get_properties_for)(OBJ_FOR_PROP(val), ZEND_PROP_PURPOSE_ARRAY_CAST); - free_ht = 1; - } else { - ht = Z_OBJPROP_P(val); -@@ -409,7 +409,7 @@ static inline void msgpack_serialize_object(smart_str *buf, zval *val, HashTable - - if (ce && ce != PHP_IC_ENTRY && - zend_hash_exists(&ce->function_table, sleep_zstring)) { -- if ((res = call_user_function_ex(CG(function_table), val_noref, &fname, &retval, 0, 0, 1, NULL)) == SUCCESS) { -+ if ((res = call_user_function(CG(function_table), val_noref, &fname, &retval, 0, 0)) == SUCCESS) { - - if (EG(exception)) { - zval_ptr_dtor(&retval); -diff --git a/msgpack_unpack.c b/msgpack_unpack.c -index 9ce1961..a602c52 100644 ---- a/msgpack_unpack.c -+++ b/msgpack_unpack.c -@@ -282,7 +282,7 @@ static zend_class_entry* msgpack_unserialize_class(zval **container, zend_string - ZVAL_STRING(&user_func, PG(unserialize_callback_func)); - ZVAL_STR(&args[0], class_name); - -- func_call_status = call_user_function_ex(CG(function_table), NULL, &user_func, &retval, 1, args, 0, NULL); -+ func_call_status = call_user_function(CG(function_table), NULL, &user_func, &retval, 1, args); - zval_ptr_dtor(&user_func); - if (func_call_status != SUCCESS) { - MSGPACK_WARNING("[msgpack] (%s) defined (%s) but not found", -@@ -329,7 +329,11 @@ static zend_class_entry* msgpack_unserialize_class(zval **container, zend_string - - /* store incomplete class name */ - if (incomplete_class) { -+#if PHP_VERSION_ID < 80000 - php_store_class_name(container_val, ZSTR_VAL(class_name), ZSTR_LEN(class_name)); -+#else -+ php_store_class_name(container_val, class_name); -+#endif - } - - return ce; -@@ -836,7 +840,7 @@ int msgpack_unserialize_map_item(msgpack_unserialize_data *unpack, zval **contai - zend_hash_str_exists(&Z_OBJCE_P(container_val)->function_table, "__wakeup", sizeof("__wakeup") - 1)) { - zval wakeup, r; - ZVAL_STRING(&wakeup, "__wakeup"); -- call_user_function_ex(CG(function_table), container_val, &wakeup, &r, 0, NULL, 1, NULL); -+ call_user_function(CG(function_table), container_val, &wakeup, &r, 0, NULL); - zval_ptr_dtor(&r); - zval_ptr_dtor(&wakeup); - } -diff --git a/php_msgpack.h b/php_msgpack.h -index d5f9164..6ecd5f4 100644 ---- a/php_msgpack.h -+++ b/php_msgpack.h -@@ -52,4 +52,10 @@ PHP_MSGPACK_API int php_msgpack_unserialize( - # define MSGPACK_ENDIAN_BIG_BYTE 0 - #endif - -+#if PHP_VERSION_ID < 80000 -+# define OBJ_FOR_PROP(zv) (zv) -+#else -+# define OBJ_FOR_PROP(zv) Z_OBJ_P(zv) -+#endif -+ - #endif /* PHP_MSGPACK_H */ -From 77368369d8c9b7a9f135d3b6aef2c26656dd4b55 Mon Sep 17 00:00:00 2001 -From: Michael Wallner <mike@php.net> -Date: Wed, 23 Sep 2020 09:13:30 +0200 -Subject: [PATCH] msgpack_convert: fix crash on callback with PHP 8 - ---- - msgpack_convert.c | 11 +++++------ - 1 file changed, 5 insertions(+), 6 deletions(-) - -diff --git a/msgpack_convert.c b/msgpack_convert.c -index 8f9e1d0..1f90ef4 100644 ---- a/msgpack_convert.c -+++ b/msgpack_convert.c -@@ -296,28 +296,27 @@ int msgpack_convert_object(zval *return_value, zval *tpl, zval *value) /* {{{ */ - - /* Run the constructor if there is one */ - if (ce->constructor && (ce->constructor->common.fn_flags & ZEND_ACC_PUBLIC)) { -- zval retval, params, function_name; -+ zval retval; - zend_fcall_info fci; - zend_fcall_info_cache fcc; - -+ memset(&fci, 0, sizeof(fci)); -+ memset(&fcc, 0, sizeof(fcc)); -+ - fci.size = sizeof(fci); - #if PHP_VERSION_ID < 70100 - fci.function_table = EG(function_table); -- fci.symbol_table = NULL; - #endif -- fci.function_name = function_name; - fci.object = Z_OBJ_P(return_value); - fci.retval = &retval; -- fci.param_count = 0; -- fci.params = ¶ms; - #if PHP_VERSION_ID < 80000 - fci.no_separation = 1; - #endif -+ - #if PHP_VERSION_ID < 70300 - fcc.initialized = 1; - #endif - fcc.function_handler = ce->constructor; -- - #if PHP_VERSION_ID < 70100 - fcc.calling_scope = EG(scope); - #else -From 87e0424e81e9187e39b3d1e7e4d1bb47a94d95e0 Mon Sep 17 00:00:00 2001 -From: Michael Wallner <mike@php.net> -Date: Wed, 23 Sep 2020 09:36:34 +0200 -Subject: [PATCH] tests: do not depend on ext/curl for a simple resource - ---- - tests/023.phpt | 18 ++---------------- - 1 file changed, 2 insertions(+), 16 deletions(-) - -diff --git a/tests/023.phpt b/tests/023.phpt -index 442fe3d..c1c72c3 100644 ---- a/tests/023.phpt -+++ b/tests/023.phpt -@@ -19,24 +19,10 @@ function test($type, $variable, $test) { - echo $test || $unserialized === null ? 'OK' : 'FAIL', PHP_EOL; - } - --if (function_exists('curl_init')) { -- $test = 'curl'; -- $res = curl_init('http://php.net/'); --} else { -- $test = 'dir'; -- $res = opendir('/tmp'); --} -- -+$res = opendir('/tmp'); - test('resource', $res, false); -+closedir($res); - --switch ($test) { -- case 'curl': -- curl_close($res); -- break; -- default: -- closedir($res); -- break; --} - ?> - --EXPECT-- - resource -From 80d82ac7b21b660501886c5b29ea1186e025de91 Mon Sep 17 00:00:00 2001 -From: Michael Wallner <mike@php.net> -Date: Wed, 23 Sep 2020 09:53:29 +0200 -Subject: [PATCH] tests: PHP 8 somehow detects recursion earlier - ---- - package.xml | 242 ++++++++++++++++++++++++------------------------ - tests/009.phpt | 7 +- - tests/009a.phpt | 99 ++++++++++++++++++++ - tests/026.phpt | 7 +- - tests/026a.phpt | 109 ++++++++++++++++++++++ - 5 files changed, 336 insertions(+), 128 deletions(-) - create mode 100644 tests/009a.phpt - create mode 100644 tests/026a.phpt - -diff --git a/tests/009.phpt b/tests/009.phpt -index feed76f..b917e98 100644 ---- a/tests/009.phpt -+++ b/tests/009.phpt -@@ -2,11 +2,10 @@ - Check for reference serialization - --SKIPIF-- - <?php --if ((version_compare(PHP_VERSION, '5.2.13') <= 0) || -- (version_compare(PHP_VERSION, '5.3.0') >= 0 && -- version_compare(PHP_VERSION, '5.3.2') <= 0)) { -- echo "skip tests in PHP 5.2.14/5.3.3 or newer"; -+if (version_compare(PHP_VERSION, "8.0.0dev", ">=")) { -+ die("skip PHP >= 8.0\n"); - } -+?> - --FILE-- - <?php - if(!extension_loaded('msgpack')) { -diff --git a/tests/009a.phpt b/tests/009a.phpt -new file mode 100644 -index 0000000..a155604 ---- /dev/null -+++ b/tests/009a.phpt -@@ -0,0 +1,99 @@ -+--TEST-- -+Check for reference serialization -+--SKIPIF-- -+<?php -+if (version_compare(PHP_VERSION, "8.0.0dev", "<")) { -+ die("skip PHP < 8.0\n"); -+} -+?> -+--FILE-- -+<?php -+if(!extension_loaded('msgpack')) { -+ dl('msgpack.' . PHP_SHLIB_SUFFIX); -+} -+ -+function test($type, $variable, $test) { -+ $serialized = msgpack_serialize($variable); -+ $unserialized = msgpack_unserialize($serialized); -+ -+ echo $type, PHP_EOL; -+ echo bin2hex($serialized), PHP_EOL; -+ var_dump($unserialized); -+ echo $test || $unserialized == $variable ? 'OK' : 'ERROR', PHP_EOL; -+} -+ -+$a = array('foo'); -+ -+test('array($a, $a)', array($a, $a), false); -+test('array(&$a, &$a)', array(&$a, &$a), false); -+ -+$a = array(null); -+$b = array(&$a); -+$a[0] = &$b; -+ -+test('cyclic', $a, true); -+ -+var_dump($a); -+var_dump(msgpack_unserialize(msgpack_serialize($a))); -+ -+--EXPECT-- -+array($a, $a) -+9291a3666f6f91a3666f6f -+array(2) { -+ [0]=> -+ array(1) { -+ [0]=> -+ string(3) "foo" -+ } -+ [1]=> -+ array(1) { -+ [0]=> -+ string(3) "foo" -+ } -+} -+OK -+array(&$a, &$a) -+9282c00100a3666f6f82c0020002 -+array(2) { -+ [0]=> -+ &array(1) { -+ [0]=> -+ string(3) "foo" -+ } -+ [1]=> -+ &array(1) { -+ [0]=> -+ string(3) "foo" -+ } -+} -+OK -+cyclic -+9182c0010082c0010082c0020002 -+array(1) { -+ [0]=> -+ &array(1) { -+ [0]=> -+ array(1) { -+ [0]=> -+ *RECURSION* -+ } -+ } -+} -+OK -+array(1) { -+ [0]=> -+ &array(1) { -+ [0]=> -+ *RECURSION* -+ } -+} -+array(1) { -+ [0]=> -+ &array(1) { -+ [0]=> -+ array(1) { -+ [0]=> -+ *RECURSION* -+ } -+ } -+} -diff --git a/tests/026.phpt b/tests/026.phpt -index d89654d..ba59a71 100644 ---- a/tests/026.phpt -+++ b/tests/026.phpt -@@ -3,11 +3,10 @@ Cyclic array test - --INI-- - --SKIPIF-- - <?php --if ((version_compare(PHP_VERSION, '5.2.13') <= 0) || -- (version_compare(PHP_VERSION, '5.3.0') >= 0 && -- version_compare(PHP_VERSION, '5.3.2') <= 0)) { -- echo "skip tests in PHP 5.2.14/5.3.3 or newer"; -+if (version_compare(PHP_VERSION, "8.0.0dev", ">=")) { -+ die("skip PHP >= 8.0"); - } -+?> - --FILE-- - <?php - if(!extension_loaded('msgpack')) { -diff --git a/tests/026a.phpt b/tests/026a.phpt -new file mode 100644 -index 0000000..8abc61a ---- /dev/null -+++ b/tests/026a.phpt -@@ -0,0 +1,109 @@ -+--TEST-- -+Cyclic array test -+--INI-- -+--SKIPIF-- -+<?php -+ -+if (version_compare(PHP_VERSION, "8.0.0dev", "<")) { -+ die("skip PHP < 8.0"); -+} -+?> -+--FILE-- -+<?php -+if(!extension_loaded('msgpack')) { -+ dl('msgpack.' . PHP_SHLIB_SUFFIX); -+} -+ -+function test($type, $variable, $test) { -+ $serialized = msgpack_serialize($variable); -+ $unserialized = msgpack_unserialize($serialized); -+ -+ echo $type, PHP_EOL; -+ echo bin2hex($serialized), PHP_EOL; -+ var_dump($unserialized); -+ echo $test || $unserialized == $variable ? 'OK' : 'ERROR', PHP_EOL; -+} -+ -+$a = array( -+ 'a' => array( -+ 'b' => 'c', -+ 'd' => 'e' -+ ), -+); -+ -+$a['f'] = &$a; -+ -+test('array', $a, true); -+ -+$a = array("foo" => &$b); -+$b = array(1, 2, $a); -+var_dump($a); -+var_dump($k = msgpack_unserialize(msgpack_serialize($a))); -+ -+$k["foo"][1] = "b"; -+var_dump($k); -+?> -+--EXPECT-- -+array -+82a16182a162a163a164a165a16683c001a16182a162a163a164a165a16682c0020003 -+array(2) { -+ ["a"]=> -+ array(2) { -+ ["b"]=> -+ string(1) "c" -+ ["d"]=> -+ string(1) "e" -+ } -+ ["f"]=> -+ &array(2) { -+ ["a"]=> -+ array(2) { -+ ["b"]=> -+ string(1) "c" -+ ["d"]=> -+ string(1) "e" -+ } -+ ["f"]=> -+ *RECURSION* -+ } -+} -+OK -+array(1) { -+ ["foo"]=> -+ &array(3) { -+ [0]=> -+ int(1) -+ [1]=> -+ int(2) -+ [2]=> -+ *RECURSION* -+ } -+} -+array(1) { -+ ["foo"]=> -+ &array(3) { -+ [0]=> -+ int(1) -+ [1]=> -+ int(2) -+ [2]=> -+ array(1) { -+ ["foo"]=> -+ *RECURSION* -+ } -+ } -+} -+array(1) { -+ ["foo"]=> -+ &array(3) { -+ [0]=> -+ int(1) -+ [1]=> -+ string(1) "b" -+ [2]=> -+ array(1) { -+ ["foo"]=> -+ *RECURSION* -+ } -+ } -+} diff --git a/php-pecl-msgpack.spec b/php-pecl-msgpack.spec index 7fd8f65..7f4da45 100644 --- a/php-pecl-msgpack.spec +++ b/php-pecl-msgpack.spec @@ -3,55 +3,47 @@ # # Fedora spec file for php-pecl-msgpack # -# Copyright (c) 2012-2020 Remi Collet -# License: CC-BY-SA +# Copyright (c) 2012-2024 Remi Collet +# License: CC-BY-SA-4.0 # http://creativecommons.org/licenses/by-sa/4.0/ # # Please, preserve the changelog entries # -%if 0%{?scl:1} -%global sub_prefix %{scl_prefix} -%scl_package php-pecl-msgpack -%endif +%{?scl:%scl_package php-pecl-msgpack} %global gh_commit 943d27267fbf6da6b4d225f344f4731aec0c671b %global gh_short %(c=%{gh_commit}; echo ${c:0:7}) %global gh_owner msgpack %global gh_project msgpack-php -#global gh_date 20171026 %global pecl_name msgpack %global with_zts 0%{!?_without_zts:%{?__ztsphp:1}} %global ini_name 40-%{pecl_name}.ini -%global upstream_version 2.1.1 -#global upstream_prever beta1 -#global upstream_lower beta1 +%global upstream_version 3.0.0 +#global upstream_prever RC1 +%global sources %{pecl_name}-%{upstream_version}%{?upstream_prever} +%global _configure ../%{sources}/configure -%if 0 -%global with_msgpack 1 -%else -%global with_msgpack 0 -%endif +# to use system library +%bcond_with msgpack + +# to disable test suite +%bcond_without tests Summary: API for communicating with MessagePack serialization -Name: %{?sub_prefix}php-pecl-msgpack -License: BSD -Version: %{upstream_version}%{?upstream_lower:~%{upstream_lower}} +Name: %{?scl_prefix}php-pecl-msgpack +License: BSD-3-Clause +Version: %{upstream_version}%{?upstream_prever:~%{upstream_prever}} URL: https://pecl.php.net/package/msgpack -%if 0%{?gh_date:1} -Release: 0.6.%{gh_date}.%{gh_short}%{?dist}%{!?scl:%{!?nophptag:%(%{__php} -r 'echo ".".PHP_MAJOR_VERSION.".".PHP_MINOR_VERSION;')}} -Source0: https://github.com/%{gh_owner}/%{gh_project}/archive/%{gh_commit}/%{pecl_name}-%{version}-%{gh_short}.tar.gz -%else -Release: 3%{?dist}%{!?scl:%{!?nophptag:%(%{__php} -r 'echo ".".PHP_MAJOR_VERSION.".".PHP_MINOR_VERSION;')}} +Release: 1%{?dist}%{!?scl:%{!?nophptag:%(%{__php} -r 'echo ".".PHP_MAJOR_VERSION.".".PHP_MINOR_VERSION;')}} Source: https://pecl.php.net/get/%{pecl_name}-%{upstream_version}%{?upstream_prever}.tgz -%endif - -Patch0: %{pecl_name}-php8.patch +BuildRequires: make BuildRequires: %{?dtsprefix}gcc -BuildRequires: %{?scl_prefix}php-devel >= 7 +BuildRequires: %{?scl_prefix}php-devel >= 7.0 BuildRequires: %{?scl_prefix}php-pear -%if %{with_msgpack} +BuildRequires: %{?scl_prefix}php-pecl-apcu-devel +%if %{with msgpack} BuildRequires: msgpack-devel %else Provides: bundled(msgpack) = 3.2.0 @@ -59,34 +51,11 @@ Provides: bundled(msgpack) = 3.2.0 Requires: %{?scl_prefix}php(zend-abi) = %{php_zend_api} Requires: %{?scl_prefix}php(api) = %{php_core_api} -%{?_sclreq:Requires: %{?scl_prefix}runtime%{?_sclreq}%{?_isa}} Provides: %{?scl_prefix}php-%{pecl_name} = %{version} Provides: %{?scl_prefix}php-%{pecl_name}%{?_isa} = %{version} Provides: %{?scl_prefix}php-pecl(%{pecl_name}) = %{version} Provides: %{?scl_prefix}php-pecl(%{pecl_name})%{?_isa} = %{version} -%if "%{?scl_prefix}" != "%{?sub_prefix}" -Provides: %{?scl_prefix}php-pecl-%{pecl_name} = %{version}-%{release} -Provides: %{?scl_prefix}php-pecl-%{pecl_name}%{?_isa} = %{version}-%{release} -%endif -%if "%{?packager}" == "Remi Collet" && 0%{!?scl:1} && 0%{?rhel} -# Other third party repo stuff -%if "%{php_version}" > "7.3" -Obsoletes: php73-pecl-%{pecl_name} <= %{version} -%endif -%if "%{php_version}" > "7.4" -Obsoletes: php74-pecl-%{pecl_name} <= %{version} -%endif -%if "%{php_version}" > "8.0" -Obsoletes: php80-pecl-%{pecl_name} <= %{version} -%endif -%endif - -%if 0%{?fedora} < 20 && 0%{?rhel} < 7 -# Filter shared private -%{?filter_provides_in: %filter_provides_in %{_libdir}/.*\.so$} -%{?filter_setup} -%endif %description @@ -102,8 +71,6 @@ If you ever wished to use JSON for convenience (storing an image with metadata) but could not for technical reasons (encoding, size, speed...), MessagePack is a perfect replacement. -This extension is still EXPERIMENTAL. - Package built for PHP %(%{__php} -r 'echo PHP_MAJOR_VERSION.".".PHP_MINOR_VERSION;')%{?scl: as Software Collection (%{scl} by %{?scl_vendor}%{!?scl_vendor:rh})}. @@ -112,10 +79,6 @@ Summary: MessagePack developer files (header) Group: Development/Libraries Requires: %{name}%{?_isa} = %{version}-%{release} Requires: %{?scl_prefix}php-devel%{?_isa} -%if "%{?scl_prefix}" != "%{?sub_prefix}" -Provides: %{?scl_prefix}php-pecl-%{pecl_name}-devel = %{version}-%{release} -Provides: %{?scl_prefix}php-pecl-%{pecl_name}-devel%{?_isa} = %{version}-%{release} -%endif %description devel @@ -124,20 +87,11 @@ These are the files needed to compile programs using MessagePack serializer. %prep %setup -qc -%if 0%{?gh_date:1} -mv %{gh_project}-%{gh_commit} NTS -mv NTS/package.xml . -sed -e '/release/s/0.5.6/%{version}%{?gh_date:dev}/' -i package.xml -%else -mv %{pecl_name}-%{upstream_version}%{?upstream_prever} NTS -%endif - -%{?_licensedir:sed -e '/LICENSE/s/role="doc"/role="src"/' -i package.xml} -cd NTS -%patch0 -p1 -b .up +sed -e '/LICENSE/s/role="doc"/role="src"/' -i package.xml -%if %{with_msgpack} +cd %{sources} +%if %{with msgpack} # use system library rm -rf msgpack %endif @@ -150,9 +104,9 @@ if test "x${extver}" != "x%{upstream_version}%{?upstream_prever}%{?gh_date:-dev} fi cd .. +mkdir NTS %if %{with_zts} -# duplicate for ZTS build -cp -pr NTS ZTS +mkdir ZTS %endif # Drop in the bit of configuration @@ -163,45 +117,50 @@ extension = %{pecl_name}.so ; Configuration options ;msgpack.error_display = On -;msgpack.illegal_key_insert = Off ;msgpack.php_only = On +;msgpack.assoc = On +;msgpack.illegal_key_insert = Off +;msgpack.use_str8_serialization = On EOF %build %{?dtsenable} -cd NTS -%{_bindir}/phpize -%configure --with-php-config=%{_bindir}/php-config -make %{?_smp_mflags} +cd %{sources} +%{__phpize} +[ -f Makefile.global ] && GLOBAL=Makefile.global || GLOBAL=build/Makefile.global +sed -e 's/INSTALL_ROOT/DESTDIR/' -i $GLOBAL + +cd ../NTS +%configure --with-php-config=%{__phpconfig} +%make_build %if %{with_zts} cd ../ZTS -%{_bindir}/zts-phpize -%configure --with-php-config=%{_bindir}/zts-php-config -make %{?_smp_mflags} +%configure --with-php-config=%{__ztsphpconfig} +%make_build %endif %install %{?dtsenable} -# Install the NTS stuff -make -C NTS install INSTALL_ROOT=%{buildroot} +: Install the NTS stuff +%make_install -C NTS install -D -m 644 %{ini_name} %{buildroot}%{php_inidir}/%{ini_name} %if %{with_zts} -# Install the ZTS stuff -make -C ZTS install INSTALL_ROOT=%{buildroot} +: Install the ZTS stuff +%make_install -C ZTS install -D -m 644 %{ini_name} %{buildroot}%{php_ztsinidir}/%{ini_name} %endif -# Install the package XML file +: Install the package XML file install -D -m 644 package.xml %{buildroot}%{pecl_xmldir}/%{name}.xml -# Test & Documentation -cd NTS +: install Test and Documentation +cd %{sources} for i in $(grep 'role="test"' ../package.xml | sed -e 's/^.*name="//;s/".*$//') do [ -f tests/$i ] && install -Dpm 644 tests/$i %{buildroot}%{pecl_testdir}/%{pecl_name}/tests/$i [ -f $i ] && install -Dpm 644 $i %{buildroot}%{pecl_testdir}/%{pecl_name}/$i @@ -214,58 +173,46 @@ done %check # Erratic results, segfauilt and mark as XFAIL rm */tests/034.phpt +%ifarch aarch64 +# too slow +rm */tests/035.phpt +%endif + +%if "%{php_version}" > "7.4" +OPT="-q -P --show-diff %{?_smp_mflags}" +%else +OPT="-q -P --show-diff" +export REPORT_EXIT_STATUS=1 +%endif -cd NTS +cd %{sources} : Minimal load test for NTS extension %{__php} --no-php-ini \ --define extension=%{buildroot}%{php_extdir}/%{pecl_name}.so \ - --modules | grep %{pecl_name} + --modules | grep '^%{pecl_name}$' +%if %{with tests} : Upstream test suite for NTS extension -TEST_PHP_EXECUTABLE=%{__php} \ -TEST_PHP_ARGS="-n -d extension_dir=$PWD/modules -d extension=%{pecl_name}.so" \ -NO_INTERACTION=1 \ -REPORT_EXIT_STATUS=0 \ -%{__php} -n run-tests.php --show-diff +TEST_PHP_ARGS="-n -d extension=apcu.so -d extension=%{buildroot}%{php_extdir}/%{pecl_name}.so" \ +%{__php} -n run-tests.php $OPT +%endif %if %{with_zts} -cd ../ZTS : Minimal load test for ZTS extension %{__ztsphp} --no-php-ini \ --define extension=%{buildroot}%{php_ztsextdir}/%{pecl_name}.so \ - --modules | grep %{pecl_name} + --modules | grep '^%{pecl_name}$' +%if %{with tests} : Upstream test suite for ZTS extension -TEST_PHP_EXECUTABLE=%{__ztsphp} \ -TEST_PHP_ARGS="-n -d extension_dir=$PWD/modules -d extension=%{pecl_name}.so" \ -NO_INTERACTION=1 \ -REPORT_EXIT_STATUS=0 \ -%{__ztsphp} -n run-tests.php --show-diff +TEST_PHP_ARGS="-n -d extension=apcu.so -d extension=%{buildroot}%{php_ztsextdir}/%{pecl_name}.so" \ +%{__ztsphp} -n run-tests.php $OPT %endif - - -%if 0%{?fedora} < 24 && 0%{?rhel} < 8 -# when pear installed alone, after us -%triggerin -- %{?scl_prefix}php-pear -if [ -x %{__pecl} ] ; then - %{pecl_install} %{pecl_xmldir}/%{name}.xml >/dev/null || : -fi - -# posttrans as pear can be installed after us -%posttrans -if [ -x %{__pecl} ] ; then - %{pecl_install} %{pecl_xmldir}/%{name}.xml >/dev/null || : -fi - -%postun -if [ $1 -eq 0 -a -x %{__pecl} ] ; then - %{pecl_uninstall} %{pecl_name} >/dev/null || : -fi %endif %files -%{?_licensedir:%license NTS/LICENSE} +%license %{sources}/LICENSE %doc %{pecl_docdir}/%{pecl_name} %{pecl_xmldir}/%{name}.xml @@ -288,6 +235,57 @@ fi %changelog +* Thu Sep 26 2024 Remi Collet <remi@remirepo.net> - 3.0.0-1 +- update to 3.0.0 + +* Tue Sep 24 2024 Remi Collet <remi@remirepo.net> - 3.0.0~RC1-2 +- rebuild for 8.4.0RC1 + +* Mon Aug 26 2024 Remi Collet <remi@remirepo.net> - 3.0.0~RC1-1 +- update to 3.0.0RC1 +- drop patch merged upstream +- add APCu support + +* Fri Jul 5 2024 Remi Collet <remi@remirepo.net> - 2.2.0-3 +- fix test suite with 8.4 using patch from + https://github.com/msgpack/msgpack-php/pull/174 + +* Wed Aug 30 2023 Remi Collet <remi@remirepo.net> - 2.2.0-2 +- rebuild for PHP 8.3.0RC1 + +* Fri Jun 2 2023 Remi Collet <remi@remirepo.net> - 2.2.0-1 +- update to 2.2.0 +- build out of sources tree +- use parallel execution for test suite + +* Fri Oct 7 2022 Remi Collet <remi@remirepo.net> - 2.2.0~RC2-1 +- update to 2.2.0RC2 + +* Fri Sep 9 2022 Remi Collet <remi@remirepo.net> - 2.2.0~RC1-8 +- ignore 8 failed tests with PHP 8.2 + +* Thu Jul 28 2022 Remi Collet <remi@remirepo.net> - 2.2.0~RC1-7 +- skip one test on aarch64 + +* Mon Sep 6 2021 Remi Collet <remi@remirepo.net> - 2.2.0~RC1-6 +- F35 build + +* Wed Sep 01 2021 Remi Collet <remi@remirepo.net> - 2.2.0~RC1-4 +- rebuild for 8.1.0RC1 + +* Tue Aug 31 2021 Remi Collet <remi@remirepo.net> - 2.2.0~RC1-1 +- update to 2.2.0RC1 + +* Thu Jun 10 2021 Remi Collet <remi@remirepo.net> - 2.1.2-3 +- add patch for test suite with PHP 8.1 from + https://github.com/msgpack/msgpack-php/pull/156 + +* Wed Apr 28 2021 Remi Collet <remi@remirepo.net> - 2.1.2-2 +- F34 rebuild for https://github.com/remicollet/remirepo/issues/174 + +* Sat Nov 28 2020 Remi Collet <remi@remirepo.net> - 2.1.2-1 +- update to 2.1.2 + * Wed Sep 30 2020 Remi Collet <remi@remirepo.net> - 2.1.1-3 - rebuild for PHP 8.0.0RC1 |