Martin Panter added the comment:

The ABI matching looks wrong to me. If I am looking for a 32-bit library, won’t 
it incorrectly catch the wrong path in the following “ldconfig -p” output:

'\tlibm.so.6 (libc6,x86-64, OS ABI: Linux 2.6.32) => /usr/lib/libm.so.6\n'
'\tlibm.so.6 (libc6, OS ABI: Linux 2.6.32) => /usr/lib32/libm.so.6\n'

Perhaps the abi_type check needs to be moved in front of the path name 
extraction.

For the version, I would put 3.6. Since this changes documented behaviour and 
has the potential to break compatibilty, it is best not to change it in a bug 
fix release. (3.5 has already been released.)

The problem with the test() function in ctypes.util is that it is not run by 
the main Python regression test suite. The tests under ctypes/test/ are run by 
the test suite.

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<http://bugs.python.org/issue21042>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to