iphp icon indicating copy to clipboard operation
iphp copied to clipboard

add a "requirements" section in the readme

Open lelutin opened this issue 14 years ago • 3 comments

the readme file should list the requirements for running iphp.

So far, I can see PHP 5 >= 5.1 is needed because of use of the readline_callback_handler_install function. and, php-cli is also needed.

also, PHP must be compiled with option --with-readline

lelutin avatar Apr 20 '10 14:04 lelutin

Thanks for the info.

I guess it'd be possible to make it work without readline, but I am not sure you'd want to. It would be so painful to use... I will try to add this info to the requirements.

apinstein avatar Apr 20 '10 15:04 apinstein

I encountered a problem while trying out iphp:

right after the welcome message, it says:

Fatal error: Call to undefined function readline_callback_handler_install() in /home/me/Desktop/apinstein-iphp-9070a44/iphp.php on line 420

php -v gives 5.2.6 and php -m says the "readline" module is active.

around line 420, the if clause checks for existance of the "readline" function but uses "readline_callback_handler_install".

changing the if to test for readline_callback_handler_install instead made the script run.

after fixing this, I can see that you are using a fallback to readline_callback_handler_install so maybe the requirements are not as high as php >= 5.1 ..

lelutin avatar Apr 20 '10 18:04 lelutin

Interesting. I am not sure why I check that way -- I looked in the php docs for readline_callback_handler_install and didn't see where it was not bundled with readline.

BUt if your patch works, that's great. I will look into it in the future for fixing it in master.

apinstein avatar Apr 20 '10 19:04 apinstein