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
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
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
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-2024 The PHP Group
All rights reserved.
Last updated: Fri Mar 29 13:01:29 2024 UTC