[Twisted-Python] Bytes vs unicode in twisted.python.logfile's python3 porting

[Laurens Van Houtven]

Hi everyone,


I'm working on #6749 for porting t.p.logfile to python3. I'm dealing with
some test failures, which you can see from the buildbot here:

http://buildbot.twistedmatrix.com/builders/python-3.3-tests/builds/1602/steps/shell/logs/stdio

I have pasted the relevant bit into a gist here:

https://gist.github.com/lvh/7174766

I think what's happening is that LogFile.write should take native strings
(since that's what log.msg takes). However, I'm opening all files in binary
mode, since that's on the reviewer checklist (point 8) for the Python 3
porting plan.

Should I not open the files in binary mode?

cheers
lvh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://twistedmatrix.com/pipermail/twisted-python/attachments/20131026/94e8531f/attachment.html>