[merge] use os._exit

Martin Pool mbp at canonical.com
Mon Feb 19 04:38:43 GMT 2007


On 19 Feb 2007, Alexander Belchenko <bialix at ukr.net> wrote:
> This patch already merged.
> But I want to say that it's bad. I'm -1 on this change.
> This patch cause to sudden break of tail in .bzr.log.
> Therefore long traceback for some error is not recorded properly.

I'm sorry you didn't get to look at it first.

This failure is clearly because we are not explicitly closing that file.

In my opinion we should explicitly close the file, and that will fix
this.  There is still some risk that other things won't be covered, and
I'll understand if people just want to back this out, but at the moment
I would rather fix them case by case.

-- 
Martin



More information about the bazaar mailing list