Code review comment for lp:~songofacandy/bzr/fix-524560

Revision history for this message
Martin Packman (gz) wrote :

Okay, I think some version of this should go in.

I've got a competing branch that does the minimal 'N' flag adding here:
<https://code.launchpad.net/~gz/bzr/noinherit_file_handles_524560>

But I was under the impression that the all-in-one installer used the latest Python. As it's still on 2.5 my branch wouldn't actually fix the edge case in the bug for most people.

review: Approve

« Back to merge proposal