Marc-Andre Lemburg added the comment:

The patch looks good, except one nit:

if 'IronPython' in sys_version

is probably not supported in IronPython 2.0, since support for "n-char in 
m-char" tests were added after Python 2.1.

Now, you can either leave this in and remove the IronPython 2.0 support from 
platform.py (which is fine, IMO), or change the test to use the .find() method.

----------

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

Reply via email to