php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #57269 Fails to build, onStatement declaration error
Submitted: 2006-09-30 19:48 UTC Modified: 2015-02-26 07:35 UTC
From: ludvig dot ericson at gmail dot com Assigned:
Status: Suspended Package: apd (PECL)
PHP Version: 5.1.6 OS: Gentoo Linux 2.6
Private report: No CVE-ID: None
Have you experienced this issue?
Rate the importance of this bug to you:

 [2006-09-30 19:48 UTC] ludvig dot ericson at gmail dot com
Description:
------------
When trying to build apd-1.0.1 under Gentoo Linux, utilizing the `pecl install` it fails to compile, due to a mismatching declaration error:

gcc -I. -I/tmp/pear/cache/apd-1.0.1 -DPHP_ATOM_INC -I/var/tmp/pear-build-root/apd-1.0.1/include -I/var/tmp/pear-build-root/apd-1.0.1/main -I/tmp/pear/cache/apd-1.0.1 -I/usr/lib/php5/include/php -I/usr/li
b/php5/include/php/main -I/usr/lib/php5/include/php/TSRM -I/usr/lib/php5/include/php/Zend -I/usr/lib/php5/include/php/ext -DHAVE_CONFIG_H -g -O2 -c /tmp/pear/cache/apd-1.0.1/php_apd.c  -fPIC -DPIC -o .lib
s/php_apd.o
/tmp/pear/cache/apd-1.0.1/php_apd.c:957: error: conflicting types for 'onStatement'
/tmp/pear/cache/apd-1.0.1/php_apd.c:65: error: previous declaration of 'onStatement' was here

A patch has been included in 'Reproduce Code' which will remove the TSRMLS_DC, as I am guessing that this is what is to be removed (as opposed to what could be added to line 65)

Reproduce code:
---------------
--- apd-1.0.1/php_apd.c 2004-09-28 05:25:59.000000000 +0200
+++ apd-patch/php_apd.c 2006-10-01 01:43:12.000000000 +0200
@@ -62,7 +62,7 @@
 #endif
 ZEND_DLEXPORT void (*old_execute)(zend_op_array *op_array TSRMLS_DC);
 
-ZEND_DLEXPORT void onStatement(zend_op_array *op_array TSRMLS_DC);
+ZEND_DLEXPORT void onStatement(zend_op_array *op_array);
 ZEND_DECLARE_MODULE_GLOBALS(apd);
 
 /* This comes from php install tree. */


Expected result:
----------------
N/A

Actual result:
--------------
N/A

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2015-02-26 07:35 UTC] krakjoe@php.net
-Status: Open +Status: Suspended
 [2015-02-26 07:35 UTC] krakjoe@php.net
APD hasn't had a release in 10 years, this means it's source code is way out of sync with modern PHP.

I'm going to mark this bug as suspended, the report can still be found if a maintainer for APD comes forward.

Sorry about the wait.
 
PHP Copyright © 2001-2021 The PHP Group
All rights reserved.
Last updated: Thu Dec 02 19:03:38 2021 UTC