From: Stefan Krah Date: Wed, 16 Jun 2010 15:07:41 +0000 (+0000) Subject: Issue #7384: If the system readline library is linked against X-Git-Tag: v3.1.3rc1~629 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b0434fcd4b7bbfddc556f938e3a4eb8e0934d2d0;p=python Issue #7384: If the system readline library is linked against ncurses, do not link the readline module against ncursesw. --- diff --git a/Lib/test/test_curses.py b/Lib/test/test_curses.py index 57d4e8b161..302ddb9bda 100644 --- a/Lib/test/test_curses.py +++ b/Lib/test/test_curses.py @@ -23,11 +23,6 @@ requires('curses') curses = import_module('curses') curses.panel = import_module('curses.panel') -# skip all these tests on FreeBSD: test_curses currently hangs the -# FreeBSD buildbots, preventing other tests from running. See issue -# #7384. -if 'freebsd' in sys.platform: - raise unittest.SkipTest('The curses module is broken on FreeBSD. See http://bugs.python.org/issue7384.') # XXX: if newterm was supported we could use it instead of initscr and not exit term = os.environ.get('TERM') diff --git a/Misc/NEWS b/Misc/NEWS index ed4bc86a8f..adb9a34777 100644 --- a/Misc/NEWS +++ b/Misc/NEWS @@ -244,6 +244,11 @@ Library Extension Modules ----------------- +- Issue #7384: If the system readline library is linked against + ncurses, do not link the readline module against ncursesw. The + additional restriction of linking the readline and curses modules + against the same curses library is currently not enabled. + - Issue #8973: Add __all__ to struct module; this ensures that help(struct) includes documentation for the struct.Struct class. diff --git a/setup.py b/setup.py index 8eafeeb85d..880840d90e 100644 --- a/setup.py +++ b/setup.py @@ -14,6 +14,7 @@ from distutils.core import Extension, setup from distutils.command.build_ext import build_ext from distutils.command.install import install from distutils.command.install_lib import install_lib +from distutils.spawn import find_executable # This global variable is used to hold the list of modules to be disabled. disabled_module_list = [] @@ -538,6 +539,42 @@ class PyBuildExt(build_ext): # readline do_readline = self.compiler.find_library_file(lib_dirs, 'readline') + readline_termcap_library = "" + curses_library = "" + # Determine if readline is already linked against curses or tinfo. + if do_readline and find_executable('ldd'): + # Cannot use os.popen here in py3k. + tmpfile = os.path.join(self.build_temp, 'readline_termcap_lib') + if not os.path.exists(self.build_temp): + os.makedirs(self.build_temp) + os.system("ldd %s > %s" % (do_readline, tmpfile)) + fp = open(tmpfile) + for ln in fp: + if 'curses' in ln: + readline_termcap_library = re.sub( + r'.*lib(n?cursesw?)\.so.*', r'\1', ln + ).rstrip() + break + if 'tinfo' in ln: # termcap interface split out from ncurses + readline_termcap_library = 'tinfo' + break + fp.close() + os.unlink(tmpfile) + # Issue 7384: If readline is already linked against curses, + # use the same library for the readline and curses modules. + # Disabled since applications relying on ncursesw might break. + # + # if 'curses' in readline_termcap_library: + # curses_library = readline_termcap_library + # elif self.compiler.find_library_file(lib_dirs, 'ncursesw'): + # (...) + if self.compiler.find_library_file(lib_dirs, 'ncursesw'): + curses_library = 'ncursesw' + elif self.compiler.find_library_file(lib_dirs, 'ncurses'): + curses_library = 'ncurses' + elif self.compiler.find_library_file(lib_dirs, 'curses'): + curses_library = 'curses' + if platform == 'darwin': # and os.uname()[2] < '9.': # MacOSX 10.4 has a broken readline. Don't try to build # the readline module unless the user has installed a fixed @@ -558,14 +595,10 @@ class PyBuildExt(build_ext): readline_extra_link_args = () readline_libs = ['readline'] - if self.compiler.find_library_file(lib_dirs, - 'ncursesw'): - readline_libs.append('ncursesw') - elif self.compiler.find_library_file(lib_dirs, - 'ncurses'): - readline_libs.append('ncurses') - elif self.compiler.find_library_file(lib_dirs, 'curses'): - readline_libs.append('curses') + if readline_termcap_library: + pass # Issue 7384: Already linked against curses or tinfo. + elif curses_library: + readline_libs.append(curses_library) elif self.compiler.find_library_file(lib_dirs + ['/usr/lib/termcap'], 'termcap'): @@ -1071,19 +1104,15 @@ class PyBuildExt(build_ext): # Curses support, requiring the System V version of curses, often # provided by the ncurses library. panel_library = 'panel' - if (self.compiler.find_library_file(lib_dirs, 'ncursesw')): - curses_libs = ['ncursesw'] - # Bug 1464056: If _curses.so links with ncursesw, - # _curses_panel.so must link with panelw. - panel_library = 'panelw' - exts.append( Extension('_curses', ['_cursesmodule.c'], - libraries = curses_libs) ) - elif (self.compiler.find_library_file(lib_dirs, 'ncurses')): - curses_libs = ['ncurses'] + if curses_library.startswith('ncurses'): + if curses_library == 'ncursesw': + # Bug 1464056: If _curses.so links with ncursesw, + # _curses_panel.so must link with panelw. + panel_library = 'panelw' + curses_libs = [curses_library] exts.append( Extension('_curses', ['_cursesmodule.c'], libraries = curses_libs) ) - elif (self.compiler.find_library_file(lib_dirs, 'curses') - and platform != 'darwin'): + elif curses_library == 'curses' and platform != 'darwin': # OSX has an old Berkeley curses, not good enough for # the _curses module. if (self.compiler.find_library_file(lib_dirs, 'terminfo')):