Req #47468 [Opn->Csd]: enable readline for embed sapi

From: Date: Tue, 06 Aug 2013 09:19:49 +0000
Subject: Req #47468 [Opn->Csd]: enable readline for embed sapi
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-180756@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=47468&edit=1

 ID:                 47468
 Updated by:         yohgaki@php.net
 Reported by:        admin at mateuszjanowski dot pl
 Summary:            enable readline for embed sapi
-Status:             Open
+Status:             Closed
 Type:               Feature/Change Request
 Package:            Other web server
 Operating System:   *
 PHP Version:        5.*
-Assigned To:        
+Assigned To:        yohgaki
 Block user comment: N
 Private report:     N

 New Comment:

It seems this issue is fixed.


Previous Comments:
------------------------------------------------------------------------
[2009-04-25 15:38:26] admin at mateuszjanowski dot pl

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

Thank you :).

------------------------------------------------------------------------
[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 14:16:38] paul dot biggar at gmail dot com

Here is the context: http://www.phpcompiler.org/lists/phc-general/2009-February/000907.html

I would mark this as a bug, not a feature request. The readline functions are not available in the
embed SAPI. However, there is no reason that it should be impossible for an embed user to have
access to readline.

------------------------------------------------------------------------
[2009-02-21 17:47:26] admin at mateuszjanowski dot pl

Description:
------------
Hello, is my first bug report;

I'm using php compiler (phc - http://www.phpcompiler.org/) with php CLI [5.2.8]. I meet
first problem when i started to use readline extension. I compiled php with '--with
readline' is working ok but when i compile my program using phc i get;

'Fatal error: status(): Call to undefined function readline() in rcon.php on line 52'.

The phc author tell me to do;

"I would recommend filing a bug with PHP, asking for the readline
extension to be included in the embed SAPI. It seems like an oversight
rather than a feature. In addition, it might be straightforward to
create a patch to fix this yourself. I took a quick look at this, but
I couldn't find where to do this. You might have more luck."

Btw i compiled PHP 5.2.9RC2 version and getting the same error.

Mabye you can help me?

Greetings Mateusz Janowski.


Reproduce code:
---------------
http://mateuszjanowski.pl/rcon.phps

Expected result:
----------------
0.2 Alpha - www.torvalds.pl
Remote Console: IP:PORT...

Rcon: <You write here commands which is send to the server.>

Actual result:
--------------
When i run php -f rcon.php is working OK but after compiling nope;

0.2 Alpha - www.torvalds.pl
Remote Console: IP:PORT...


Fatal error: status(): Call to undefined function readline() in rcon.php on line 52



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



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


Thread (9 messages)

« previous php.bugs (#180756) next »