Re: [PATCH 2/2] daemon: graceful shutdown of client connection

2018-04-18 Thread Johannes Schindelin
Hi Kim, On Sun, 15 Apr 2018, Kim Gybels wrote: > On (13/04/18 15:03), Johannes Schindelin wrote: > > I wonder whether you found a reliable way to trigger this? It would be > > nice to have a regression test for this. > > On my system, it reproduced reliably using Oleg's example [1], below is >

Re: [PATCH 2/2] daemon: graceful shutdown of client connection

2018-04-15 Thread Kim Gybels
On (13/04/18 15:03), Johannes Schindelin wrote: > I wonder whether you found a reliable way to trigger this? It would be > nice to have a regression test for this. On my system, it reproduced reliably using Oleg's example [1], below is my bash version of it. Script to generate repository with

Re: [PATCH 2/2] daemon: graceful shutdown of client connection

2018-04-13 Thread Johannes Schindelin
Hi Kim, On Thu, 12 Apr 2018, Kim Gybels wrote: > On Windows, a connection is shutdown when the last open handle to it is > closed. When that last open handle is stdout of our child process, an > abortive shutdown is triggered when said process exits. Ensure a > graceful shutdown of the client

[PATCH 2/2] daemon: graceful shutdown of client connection

2018-04-12 Thread Kim Gybels
On Windows, a connection is shutdown when the last open handle to it is closed. When that last open handle is stdout of our child process, an abortive shutdown is triggered when said process exits. Ensure a graceful shutdown of the client connection by keeping an open handle until we detect our