This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author rpetrov
Recipients akuchling, mbeachy, rpetrov
Date 2008-08-27.18:53:30
SpamBayes Score 0.0007433927
Marked as misclassified No
Message-id <1219863212.75.0.107061592067.issue1204@psf.upfronthosting.co.za>
In-reply-to
Content
In the configure{.in} exist another bug:
--------------
AC_CHECK_LIB(readline, readline)
if test "$ac_cv_have_readline_readline" = no
then
  AC_CHECK_LIB(termcap, readline)
fi
--------------
but "grep _readline_readline configure" show that variable in use is
$ac_cv_lib_readline_readline, so the check for function termcap in
readline can be removed safely - it is never reached.

I would like to propose another patch that don't use possible unresolved
symbol to detect presence of library. The new patch will define
HAVE_LIBREADLINE. If necessary will check for dependent library
and link it in correct order. The script print messages like next:
------
checking how to link readline libs... -lreadline -lncursesw
checking for rl_callback_handler_install in -lreadline... yes
checking for rl_pre_input_hook in -lreadline... yes
checking for rl_completion_matches in -lreadline... yes
------

The patch is for branch release25-maint.
It is without changes in configure script, i.e autoconf has to run to
recreate it after applying.
The patch can be applied to trunk as well.
History
Date User Action Args
2008-08-27 18:53:32rpetrovsetrecipients: + rpetrov, akuchling, mbeachy
2008-08-27 18:53:32rpetrovsetmessageid: <1219863212.75.0.107061592067.issue1204@psf.upfronthosting.co.za>
2008-08-27 18:53:31rpetrovlinkissue1204 messages
2008-08-27 18:53:30rpetrovcreate