#50209 [NoF->Csd]: Compiling with libedit cannot find readline.h

ID: 50209 Updated by: felipe@php.net Reported By: tcallawa at redhat dot com -Status: No Feedback +Status: Closed Bug Type: Compile Failure Operating System: Linux (Fedora) -PHP Version: 5.3.0 +PHP Version: 5.2, 5.3, 6 -Assigned To: +Assigned To: felipe New Comment: This bug has been fixed in SVN. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Thanks for the patch! Previous Comments: ------------------------------------------------------------------------ [2009-12-13 17:06:49] svn@php.net Automatic comment from SVN on behalf of felipe Revision: http://svn.php.net/viewvc/?view=revision&revision=292081 Log: - Fixed bug #50209 (Compiling with libedit cannot find readline.h) (patch by tcallawa at redhat dot com) ------------------------------------------------------------------------ [2009-11-26 01:00:01] php-bugs at lists dot php dot net No feedback was provided for this bug for over a week, so it is being suspended automatically. If you are able to provide the information that was originally requested, please do so and change the status of the bug back to "Open". ------------------------------------------------------------------------ [2009-11-18 14:26:01] jani@php.net Please try using this snapshot: http://snaps.php.net/php5.3-latest.tar.gz For Windows: http://windows.php.net/snapshots/ ------------------------------------------------------------------------ [2009-11-17 21:49:34] tcallawa at redhat dot com Description: ------------ The PHP code assumes that even with libedit in use (instead of readline), that the header can be found in readline/readline.h. libedit installs its "readline.h" into editline/ (to avoid the obvious conflict with readline). Accordingly, configure claims that libedit is not properly installed, when it is. I've written a straightforward patch to fix this: http://spot.fedorapeople.org/php-5.3.0-libedit.patch It is also possible to use pkgconfig to ask libedit for its include flags (and cflags), which would return "-I/usr/include/editline", but as that was slightly more intrusive, I did not go down that road. In addition, on a system where readline-devel and libedit-devel are both installed, the existing code checking for /usr/include/readline-devel will cause a false positive when checking for libedit, and will end up doing a compile with readline's headers and libedit's library. Reproduce code: --------------- ./configure --without-readline --with-libedit Expected result: ---------------- checking for libedit readline replacement... yes checking for readline in -ledit... yes Actual result: -------------- checking for libedit readline replacement... yes configure: error: Please reinstall libedit - I cannot find readline.h ------------------------------------------------------------------------ -- Edit this bug report at http://bugs.php.net/?id=50209&edit=1