[merge] use os._exit

Wouter van Heyst larstiq at larstiq.dyndns.org
Mon Feb 19 14:20:28 GMT 2007


On Mon, Feb 19, 2007 at 03:38:43PM +1100, Martin Pool wrote:
> 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.

Provided there isn't a huge amount of cases, sure.

Wouter van Heyst
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 307 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20070219/088fc3a0/attachment.pgp 


More information about the bazaar mailing list