Paul Moore <p.f.mo...@gmail.com> added the comment: On 17 October 2011 14:15, Éric Araujo <rep...@bugs.python.org> wrote: >> The file passed to csv.writer should be opened with newline=''. > How will we port this to 2.x?
No idea :-( The 2.7 documentation says use the 'b' flag, but that probably doesn't allow an encoding parameter (it doesn't on 3.x). >> I don't expect the test will catch the issue except on Windows... > Do you mean that the test will fail or be a no-op on other OSes? We can mark > it as Windows-specific (@unittest.skipIf(sys.platform != 'win32', 'test only > relevant on win32')) or just let it run if it’s harmless. The important > point is: does it fail before the fix, does it pass after? The test fails before the fix, passes after. It's a no-op on platforms where text and binary files are the same, (i.e., non-Windows systems). So it's harmless. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue13175> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com