Mark Dickinson <dicki...@gmail.com> 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 <rep...@bugs.python.org>
<http://bugs.python.org/issue7384>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to