|  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
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
Block user comment
Status: Assign to:
Bug Type:
From: ludvig dot ericson at gmail dot com
New email:
PHP Version: OS:


 [2006-09-30 19:48 UTC] ludvig dot ericson at gmail dot com
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
/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 @@
 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);
 /* This comes from php install tree. */

Expected result:

Actual result:


Add a Patch

Pull Requests

Add a Pull Request


AllCommentsChangesGit/SVN commitsRelated reports
 [2015-02-26 07:35 UTC]
-Status: Open +Status: Suspended
 [2015-02-26 07:35 UTC]
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-2024 The PHP Group
All rights reserved.
Last updated: Sun May 26 09:01:32 2024 UTC