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