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.

classification
Title: small fix for setup.py
Type: Stage:
Components: Build Versions: Python 2.3
process
Status: closed Resolution: accepted
Dependencies: Superseder:
Assigned To: akuchling Nosy List: akuchling, tim1470
Priority: normal Keywords: patch

Created on 2003-07-16 04:13 by tim1470, last changed 2022-04-10 16:09 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
setup.py.patch tim1470, 2003-07-16 04:13 use -lcursses if needed for readline
setup.py.patch2 akuchling, 2003-09-25 17:52 Add /usr/ccs/lib on Unixware as well as OSF/1
setup.py.patch3 tim1470, 2003-09-25 20:02 corrections to patch2 and added openuninx8
setup.py.patch4 akuchling, 2003-10-24 12:11 Add only unixware7 and openunix8
Messages (8)
msg44283 - (view) Author: Tim Rice (tim1470) Date: 2003-07-16 04:13
On UnixWare platforms readline module doesn't build
because it needs
-lcurses. Here is a patch that corrects this. This
should also be applied
to the 2.2 tree.

here is the error.
*** WARNING: renaming "readline" since importing it
failed: dynamic linker: ./python: relocation error:
symbol not found: tputs; referenced from:
/usr/local/lib/libreadline.so.3
msg44284 - (view) Author: A.M. Kuchling (akuchling) * (Python committer) Date: 2003-09-25 17:52
Logged In: YES 
user_id=11375

There's already an earlier bit of code that adds /usr/ccs/lib on OSF/1, so 
maybe it's better to just add Unixware to that test.  Does the new attached 
patch work for you?
msg44285 - (view) Author: Tim Rice (tim1470) Date: 2003-09-25 20:02
Logged In: YES 
user_id=618097

patch2 doesn't quite worlk but it's on the right track
I've attached setup.py.patch3 that does work.
msg44286 - (view) Author: A.M. Kuchling (akuchling) * (Python committer) Date: 2003-10-23 12:15
Logged In: YES 
user_id=11375

Question: why does the second test use 'unixware' and 'openunix8', while the 
first test uses only 'unixware'?
msg44287 - (view) Author: Tim Rice (tim1470) Date: 2003-10-23 21:38
Logged In: YES 
user_id=618097

sys.platform returns unixware7  so I added ther first test.
The OpenUNIX product name has been dropped in favor of
UnixWare again.
So I don't expect any other openunixN

Now I can't remember why I added the first hunk instead of 
just 
the second hunk with s/unixware/unixware7/
msg44288 - (view) Author: A.M. Kuchling (akuchling) * (Python committer) Date: 2003-10-24 12:11
Logged In: YES 
user_id=11375

So does the attached patch work?  It just checks for openunix8 or unixware7.
msg44289 - (view) Author: Tim Rice (tim1470) Date: 2003-10-24 18:21
Logged In: YES 
user_id=618097

Yes it works fine.
msg44290 - (view) Author: A.M. Kuchling (akuchling) * (Python committer) Date: 2003-10-24 18:26
Logged In: YES 
user_id=11375

Applied (finally :) ) as rev. 1.175 of setup.py; thanks!
History
Date User Action Args
2022-04-10 16:09:59adminsetgithub: 38858
2003-07-16 04:13:08tim1470create