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
 [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

Pull Requests

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-2024 The PHP Group
All rights reserved.
Last updated: Mon Dec 02 14:01:29 2024 UTC