-
Notifications
You must be signed in to change notification settings - Fork 781
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove GNU Readline in favor of libedit #3745
Conversation
The libedit library is a non-GPL replacement for the GNU Readline library.
reference/readline/configure.xml
Outdated
To use these functions you must compile the CGI or CLI version of PHP | ||
with readline support. You need to configure PHP | ||
<option role="configure">--with-readline[=DIR]</option>. | ||
If you want to use the libedit readline replacement, configure PHP | ||
<option role="configure">--with-libedit[=DIR]</option>. | ||
<option role="configure">--with-readline</option>. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note the documentation shouldn't use personalization (i.e. using "you") maybe you could fix the text at the same time?
This feels like this should probably use some standard wording/an existing XML entity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in the next commit. I'm not sure I know what "standard wording/an existing XML entity" means here.
What is the actual status here? Apparently, the constant |
Yes, this was merged a bit too quickly. The code related PR wasn't merged to PHP 8.4 yet. Then let's merge it in PHP 8.5 or something. |
Reverted via 51779f7. I think we should go ahead with the removal of libreadline support in php-src. |
I also think it's about time. Just for the info, it will be 6 years since the initial attempt to fix this and almost 1 year since the 2nd attempt. I'm actually quite amazed how long such issues take. And there is at least one more such issue - the gdbm library. At least from my current checks. |
Someone needs to push. :)
I have no problems with dropping this one. Should probably also drop support for ancient Berkeley DB 1 - 3 (can't imagine anybody using these for years). |
This is really difficult to keep track of. |
The libedit library is a non-GPL replacement for the GNU Readline library.