<div dir="ltr"><div><div><div><div><div>Hi everyone,<br><br><br>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:<br><br><a href="http://buildbot.twistedmatrix.com/builders/python-3.3-tests/builds/1602/steps/shell/logs/stdio">http://buildbot.twistedmatrix.com/builders/python-3.3-tests/builds/1602/steps/shell/logs/stdio</a><br>

<br></div>I have pasted the relevant bit into a gist here:<br><br><a href="https://gist.github.com/lvh/7174766">https://gist.github.com/lvh/7174766</a><br><br></div>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.<br>

<br></div>Should I not open the files in binary mode?<br><br>cheers<br>lvh<br></div></div></div>