Mark Dickinson <[email protected]> added the comment:
This patch looks good to me, assuming that the buildbots are happy. I agree
that this seems like a sensible solution for now, even if it means limiting
users to ncurses rather than ncursesw.
I was initially a bit surprised that it works on OS X, since OS X doesn't have
'ldd'; but in that case the os.system call simply outputs "sh: ldd: command
not found" to stderr and (presumably) nothing to stdout; no Python exception
is raised, so it's all okay. It might be worth adding code to avoid the
os.system('ldd ...') call on OS X, just to avoid the unnecessary error message
on the console. Apart from this, I say +1 to applying the patch.
Many thanks for all the detective work!
----------
_______________________________________
Python tracker <[email protected]>
<http://bugs.python.org/issue7384>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
