Steve Dower added the comment:

Is platform.architecture() the right way to determine what platform you're 
targeting? This will tell you the current OS, but typically you want to know 
the architecture of the Python process (since you often run 32-bit processes on 
64-bit Windows). If you just want the Python process, checking 
"sys.winver.endswith('-32')" is the easiest way in 3.5 and later to see whether 
it's 32-bit Python.

Otherwise those changes look okay to me.

----------

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

Reply via email to