Brian Curtin <cur...@acm.org> added the comment:

sys.platform will be "win32" for both 32 and 64-bit Windows.

As for Cygwin, os.name is "posix" there, and sys.platform is "cygwin", so it 
should be unaffected.

The patch looks fine to me, and we do typically use sys.platform more often 
than the os.name check.

----------
nosy: +brian.curtin

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

Reply via email to