ID:               47468
 Comment by:       admin at mateuszjanowski dot pl
 Reported By:      admin at mateuszjanowski dot pl
 Status:           Open
 Bug Type:         Feature/Change Request
 Operating System: Ubuntu 8.10 Linux
 PHP Version:      5.2.8/5.2.9RC2
 New Comment:

Hi. Yes can i confirm - Paul's patch fixes the problem.

Thank you :).


Previous Comments:
------------------------------------------------------------------------

[2009-04-25 13:11:17] anil at saog dot net

Thank you very much, it works now. Before your solution I tried to
change ext/pcntl/config.m4, a line in this file contains "cli" as a
function variable;

--- PHP_NEW_EXTENSION(pcntl, pcntl.c php_signal.c, $ext_shared, cli)
+++ PHP_NEW_EXTENSION(pcntl, pcntl.c php_signal.c, $ext_shared)

but i didnt try it, this may also help.

------------------------------------------------------------------------

[2009-04-25 12:40:22] paul dot biggar at gmail dot com

I believe this also applies to ncurses, pcntl, and libedit.

Here is a patch to fix it:

--- aclocal.m4.orig     2009-04-25 13:11:57.000000000 +0100
+++ aclocal.m4  2009-04-25 13:24:50.000000000 +0100
@@ -962,7 +962,7 @@
   if test "$3" != "shared" && test "$3" != "yes" && test "$4" = "cli";
then
 dnl ---------------------------------------------- CLI static module
     [PHP_]translit($1,a-z_-,A-Z__)[_SHARED]=no
-    if test "$PHP_SAPI" = "cgi"; then
+    if test "$PHP_SAPI" = "cgi" || test "$PHP_SAPI" = "embed"; then
       PHP_ADD_SOURCES(PHP_EXT_DIR($1),$2,$ac_extra,)
       EXT_STATIC="$EXT_STATIC $1"
     else


I tested it by building PHP 5.2.6, 5.2.9 and php5.3-200902131730,
building with

./configure --enable-embed --with-readline --enable-pcntl
--with-ncurses

and running

nm .libs/libphp5.so | grep readline
(and similarly for pcntl and ncurses)

which verifies that the readline symbols are available. I also ran the
test script from Mateusz through phc, and observed that calls to
readline now succeeded.

Can the patch be considered for 5.2.10, 5.3 and HEAD?

------------------------------------------------------------------------

[2009-02-23 19:41:07] admin at mateuszjanowski dot pl

I'm sad that you have changed category name to "Change Request". I
think, that will make it very low priority ?

------------------------------------------------------------------------

[2009-02-23 16:44:52] j...@php.net

This is not a bug but feature/change request, emphasis on "change".

------------------------------------------------------------------------

[2009-02-23 16:12:09] admin at mateuszjanowski dot pl

Category and Version updated.

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
    http://bugs.php.net/47468

-- 
Edit this bug report at http://bugs.php.net/?id=47468&edit=1

Reply via email to