STINNER Victor <victor.stin...@haypocalc.com> added the comment: > Your patch looks fine to me, except for this: > - if (platform in ('linux2', 'freebsd4', 'freebsd5', 'freebsd6', > - 'freebsd7', 'freebsd8') > - or platform.startswith("gnukfreebsd")): > + if os.uname()[0] in ('Linux', 'FreeBSD'): > > Why not use platform.system(), to be consistent?
I'm not sure that thp platform module can be used in setup.py (bootstrap issue?). It should be tested. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue12326> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com