|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
[2001-12-24 04:25 UTC] hps at intermeta dot de
[2001-12-24 04:26 UTC] hps at intermeta dot de
[2002-12-09 10:13 UTC] koce at gmx dot net
|
|||||||||||||||||||||||||||||||||||||
Copyright © 2001-2025 The PHP GroupAll rights reserved. |
Last updated: Mon Oct 20 19:00:01 2025 UTC |
I'm using cyrus-imapd 2.0.16 for E-Mail. I wanted to compile the cyrus support into php for accessing the cyrus server administartion. Compilation fails with various errors. Reason for this is simple: The name of one constant is wrong, there is a superflous _ in a constant registration and finally a void function is used in an if(). This module has never compiled (or even tested) by anyone of the PHP team. Unfortunately, fixing this bug changes the name of one registered constant in php. So a doc change may be needed, too. Patch to compile the cyrus module is attached --- php-4.1.0/ext/cyrus/cyrus.c Fri Oct 12 01:51:14 2001 +++ php-4.1.0.p/ext/cyrus/cyrus.c Sun Dec 23 14:36:22 2001 @@ -85,9 +85,9 @@ le_cyrus = zend_register_list_destructors_ex(cyrus_free, NULL, le_cyrus_name, module_number); - REGISTER_LONG_CONSTANT("CYRUS_CONN_NOSYNCLITERAL", - IMCLIENT_CONN_NOSYNCLITERAL, - CONST_CS_ | CONST_PERSISTENT); + REGISTER_LONG_CONSTANT("CYRUS_CONN_NONSYNCLITERAL", + IMCLIENT_CONN_NONSYNCLITERAL, + CONST_CS | CONST_PERSISTENT); REGISTER_LONG_CONSTANT("CYRUS_CONN_INITIALRESPONSE", IMCLIENT_CONN_INITIALRESPONSE, CONST_CS | CONST_PERSISTENT); @@ -464,8 +464,7 @@ ZEND_FETCH_RESOURCE(conn, php_cyrus *, z_conn, -1, le_cyrus_name, le_cyrus); convert_to_string_ex(query); - if (imclient_send(conn->client, NULL, NULL, Z_STRVAL_PP(query)) != 0) - RETURN_FALSE; + imclient_send(conn->client, NULL, NULL, Z_STRVAL_PP(query));