Stefan Krah <stefan-use...@bytereef.org> added the comment:

Ok, here's a more comprehensive test. Comments:

  1) Instead of emulating 'which' one could use find_executable
     from distutils.spawn. But this feels wrong.

  2) Skip the test if the primary issue cannot be tested.

  3) Exercise the test for /bin/sh, if it isn't a symlink. Then
     this part will be executed on some systems, e.g. FreeBSD.

  4) The check os.path.isfile('/bin/sh') isn't necessary, but good
     practice nonetheless.



Should this go into 2.6 and 3.1 as well?

----------
Added file: http://bugs.python.org/file18027/issue9265-2.patch

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

Reply via email to