diff options
author | Remi Collet <fedora@famillecollet.com> | 2017-02-13 16:06:46 +0100 |
---|---|---|
committer | Remi Collet <fedora@famillecollet.com> | 2017-02-13 16:06:46 +0100 |
commit | f6a4cd1a29245189ff07f6e71eed4bd3d517ca04 (patch) | |
tree | aa835b2cd910d5b3b334c8d444bbfeda94c0a32c | |
parent | 683dcb7cc060506161cf80cfea4337e46e67bd59 (diff) |
php-pecl-apm: 2.1.2
-rw-r--r-- | APM-pr44.patch | 149 | ||||
-rw-r--r-- | REFLECTION | 2 | ||||
-rw-r--r-- | php-pecl-apm.spec | 15 |
3 files changed, 11 insertions, 155 deletions
diff --git a/APM-pr44.patch b/APM-pr44.patch deleted file mode 100644 index c3256c5..0000000 --- a/APM-pr44.patch +++ /dev/null @@ -1,149 +0,0 @@ -From e4dd42a5ed8d410fd718f53d9f3f72bd91b4b0b1 Mon Sep 17 00:00:00 2001 -From: Remi Collet <fedora@famillecollet.com> -Date: Sat, 23 Jul 2016 13:45:43 +0200 -Subject: [PATCH 1/3] Fix build with PHP 7.1 - -- zend_print_zval_r_ex have been removed from 7.1 -- zend_print_zval_r_to_str have been added in 7.1 -- make apm_write static (not used anywhere else) -- move extract_data declaration ---- - apm.c | 16 +++++++++++++++- - php_apm.h | 10 +--------- - 2 files changed, 16 insertions(+), 10 deletions(-) - -diff --git a/apm.c b/apm.c -index 624369c..e45200f 100644 ---- a/apm.c -+++ b/apm.c -@@ -94,7 +94,8 @@ static int apm_end_silence_opcode_handler(ZEND_USER_OPCODE_HANDLER_ARGS) - return ZEND_USER_OPCODE_DISPATCH; - } - --int apm_write(const char *str, -+#if PHP_VERSION_ID < 70100 -+static int apm_write(const char *str, - #if PHP_VERSION_ID >= 70000 - size_t - #else -@@ -107,6 +108,7 @@ length) - smart_str_0(APM_G(buffer)); - return length; - } -+#endif - - void (*old_error_cb)(int type, const char *error_filename, const uint error_lineno, const char *format, va_list args); - -@@ -596,8 +598,14 @@ void extract_data(TSRMLS_D) - zend_is_auto_global_compat("_COOKIE"); - if (FETCH_HTTP_GLOBALS(COOKIE)) { - if (Z_ARRVAL_P(tmp)->nNumOfElements > 0) { -+#if PHP_VERSION_ID >= 70100 -+ zend_string *tmpstr; -+ tmpstr = zend_print_zval_r_to_str(tmp, 0); -+ smart_str_append(&APM_RD(cookies), tmpstr); -+#else - APM_G(buffer) = &APM_RD(cookies); - zend_print_zval_r_ex(apm_write, tmp, 0 TSRMLS_CC); -+#endif - APM_RD(cookies_found) = 1; - } - } -@@ -606,8 +614,14 @@ void extract_data(TSRMLS_D) - zend_is_auto_global_compat("_POST"); - if (FETCH_HTTP_GLOBALS(POST)) { - if (Z_ARRVAL_P(tmp)->nNumOfElements > 0) { -+#if PHP_VERSION_ID >= 70100 -+ zend_string *tmpstr; -+ tmpstr = zend_print_zval_r_to_str(tmp, 0); -+ smart_str_append(&APM_RD(post_vars), tmpstr); -+#else - APM_G(buffer) = &APM_RD(post_vars); - zend_print_zval_r_ex(apm_write, tmp, 0 TSRMLS_CC); -+#endif - APM_RD(post_vars_found) = 1; - } - } -diff --git a/php_apm.h b/php_apm.h -index 4a5ee24..3fe29e3 100644 ---- a/php_apm.h -+++ b/php_apm.h -@@ -368,14 +368,6 @@ ZEND_END_MODULE_GLOBALS(apm) - # define add_assoc_long_compat(array, key, value) add_assoc_long_ex((array), (key), (sizeof(key)), (value)); - #endif - --int apm_write(const char *str, --#if PHP_VERSION_ID >= 70000 --size_t --#else --uint --#endif --length); -- -+void extract_data(TSRMLS_D); - #endif - --void extract_data(TSRMLS_D); - -From 6109488d4b35ccb0c9b56b79986e8b5375c8f1a6 Mon Sep 17 00:00:00 2001 -From: Remi Collet <fedora@famillecollet.com> -Date: Sat, 23 Jul 2016 13:55:05 +0200 -Subject: [PATCH 2/3] Declare module dependencies (only json for socket driver) - ---- - apm.c | 11 ++++++++++- - 1 file changed, 10 insertions(+), 1 deletion(-) - -diff --git a/apm.c b/apm.c -index e45200f..1b21dd0 100644 ---- a/apm.c -+++ b/apm.c -@@ -124,8 +124,17 @@ struct timeval begin_tp; - struct rusage begin_usg; - #endif - -+static const zend_module_dep apm_deps[] = { -+#ifdef APM_DRIVER_SOCKET -+ ZEND_MOD_REQUIRED("json") -+#endif -+ ZEND_MOD_END -+}; -+ - zend_module_entry apm_module_entry = { -- STANDARD_MODULE_HEADER, -+ STANDARD_MODULE_HEADER_EX, -+ NULL, -+ apm_deps, - "apm", - NULL, - PHP_MINIT(apm), - -From 95e6fd6fb69abce62b0e1c6a1d2a9f80f463a687 Mon Sep 17 00:00:00 2001 -From: Remi Collet <fedora@famillecollet.com> -Date: Sat, 23 Jul 2016 14:05:15 +0200 -Subject: [PATCH 3/3] memleak - ---- - apm.c | 2 ++ - 1 file changed, 2 insertions(+) - -diff --git a/apm.c b/apm.c -index 1b21dd0..99157b1 100644 ---- a/apm.c -+++ b/apm.c -@@ -611,6 +611,7 @@ void extract_data(TSRMLS_D) - zend_string *tmpstr; - tmpstr = zend_print_zval_r_to_str(tmp, 0); - smart_str_append(&APM_RD(cookies), tmpstr); -+ zend_string_release(tmpstr); - #else - APM_G(buffer) = &APM_RD(cookies); - zend_print_zval_r_ex(apm_write, tmp, 0 TSRMLS_CC); -@@ -627,6 +628,7 @@ void extract_data(TSRMLS_D) - zend_string *tmpstr; - tmpstr = zend_print_zval_r_to_str(tmp, 0); - smart_str_append(&APM_RD(post_vars), tmpstr); -+ zend_string_release(tmpstr); - #else - APM_G(buffer) = &APM_RD(post_vars); - zend_print_zval_r_ex(apm_write, tmp, 0 TSRMLS_CC); @@ -1,4 +1,4 @@ -Extension [ <persistent> extension #195 apm version 2.1.1 ] { +Extension [ <persistent> extension #162 apm version 2.1.2 ] { - INI { Entry [ apm.enabled <SYSTEM> ] diff --git a/php-pecl-apm.spec b/php-pecl-apm.spec index 097d3dc..f40bd37 100644 --- a/php-pecl-apm.spec +++ b/php-pecl-apm.spec @@ -37,19 +37,17 @@ Name: %{?sub_prefix}php-pecl-apm Summary: Alternative PHP Monitor -Version: 2.1.1 +Version: 2.1.2 %if 0%{?gh_date:1} Release: 7.%{gh_date}git%{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: 4%{?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;')}} Source0: http://pecl.php.net/get/%{proj_name}-%{version}.tgz %endif # Disable the extension and drivers by default Patch0: %{proj_name}-config.patch -# See https://github.com/patrickallaert/php-apm/pull/44 -Patch1: %{proj_name}-pr44.patch License: PHP Group: Development/Languages @@ -144,7 +142,6 @@ mv %{proj_name}-%{version} NTS cd NTS %patch0 -p0 -b .rpm sed -e 's:/var/php/apm/db:%{_localstatedir}/lib/php/apm/db:' -i apm.ini -%patch1 -p1 -b .pr44 : Sanity check, really often broken extver=$(sed -n '/#define PHP_APM_VERSION/{s/.* "//;s/".*$//;p}' php_apm.h) @@ -161,6 +158,8 @@ cp -pr NTS ZTS %build +%{?dtsenable} + peclconf() { %configure \ --enable-apm \ @@ -190,6 +189,7 @@ make %{?_smp_mflags} %install rm -rf %{buildroot} +%{?dtsenable} # Install the NTS stuff make -C NTS install INSTALL_ROOT=%{buildroot} @@ -278,6 +278,11 @@ fi %changelog +* Mon Feb 13 2017 Remi Collet <remi@fedoraproject.org> - 2.1.2-1 +- Update to 2.1.2 (stable) +- open https://github.com/patrickallaert/php-apm/issues/50 + PHP 5 build is broken + * Thu Dec 1 2016 Remi Collet <remi@fedoraproject.org> - 2.1.1-4 - rebuild with PHP 7.1.0 GA |