Issue 1076826: readline does not need termcap (original) (raw)

Created on 2004-12-01 15:32 by nijel, last changed 2022-04-11 14:56 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
Python-2.3.3-termcap.patch nijel,2004-12-01 15:32
Python-2.4-readline.patch nijel,2004-12-06 22:48
Messages (6)
msg47350 - (view) Author: Michal Čihař (nijel) * Date: 2004-12-01 15:32
Readline doesn't need -ltermcap (at least on some systems), so configure check should deal with this. Attached patch completely removes -ltermcap, however better might be to check both possibilities. Patch is against 2.3.3, however it applies to 2.4 also.
msg47351 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2004-12-05 18:59
Logged In: YES user_id=21627 The patch is incorrect. As you suggest yourself, readline *does* need -ltermcap on some systems; this patch breaks these systems. Rejecting the patch. If you can come up with a patch that performs an autoconf test first, please submit a new patch.
msg47352 - (view) Author: Michal Čihař (nijel) * Date: 2004-12-06 11:18
Logged In: YES user_id=192186 Okay, here is IMHO correct patch, which does more or less same as setup.py does for finding needed libraries.
msg47353 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2004-12-06 22:29
Logged In: YES user_id=21627 This patch is still incorrect. It breaks on systems where readline requires termcap, e.g. Solaris.
msg47354 - (view) Author: Michal Čihař (nijel) * Date: 2004-12-06 22:48
Logged In: YES user_id=192186 Ooops, I should better read what I write, there shouldn't be twice ncurses, once it should be termcap.... Sorry.
msg47355 - (view) Author: Martin v. Löwis (loewis) * (Python committer) Date: 2006-11-22 08:58
I think this patch is obsolete since r41945; closing it as out-of-date.
History
Date User Action Args
2022-04-11 14:56:08 admin set github: 41267
2004-12-01 15:32:12 nijel create