On 2/8/2018 4:35 AM, Daniel Shahaf wrote:
So, just to be clear, the problem is that svn/fs.py is not py3 compatible, and having the 'builtins' module under py2 merely exposes that? I.e., we have no reason to suspect a bug in the 'future' package's implementation of builtins.open() under py2.
That's my interpretation. As I said at the start of the thread, it was never clear why the internal temporary file was opened in text mode and update mode when the code was first written.