Hi Amadeusz,
sorry for the late reply. Didn't find the time to try the new patch on
that affected production VM earlier.
On Wed, Nov 06, 2019 at 09:11:08PM +0100, Amadeusz Sławiński wrote:
> > I am wondering why the zombies show up at all.
Me too.
> > They were reaped correctly on my test proce
On Wed, 6 Nov 2019 21:11:08 +0100
Amadeusz Sławiński wrote:
> On Wed, 6 Nov 2019 15:12:41 +0100
> Václav Doležal wrote:
>
> > I think this is bad solution. If some process deliberately closes stdout and
> > stderr, screen will block until that process exits.
> >
> > I am wondering why the zomb
On Wed, 6 Nov 2019 15:12:41 +0100
Václav Doležal wrote:
> I think this is bad solution. If some process deliberately closes stdout and
> stderr, screen will block until that process exits.
>
> I am wondering why the zombies show up at all. They were reaped correctly on
> my test process (code be
I think this is bad solution. If some process deliberately closes stdout and
stderr, screen will block until that process exits.
I am wondering why the zombies show up at all. They were reaped correctly on
my test process (code below). I guess there is some race with SIGCHLD handler.
Regards,
Vác
On Sun, 3 Nov 2019 00:17:21 +0100
Axel Beckert wrote:
> So thanks again for the patch and for applying it in git, too.
Well, it's not applied yet, I decided to change a bit my model of work,
to send patches to the list first and wait a bit for comments (around
one week.) If no one tells me that
Hi Amadeusz,
On Sat, Nov 02, 2019 at 10:59:44PM +0100, Amadeusz Sławiński wrote:
> When window dies we should wait for child process to die, otherwise
> there is no one to reap it.
>
> So emove WNOHANG in waitpid(), which may cause screen to not wait for
> child to die and kill window, leaving zo
When window dies we should wait for child process to die, otherwise
there is no one to reap it.
So emove WNOHANG in waitpid(), which may cause screen to not wait for
child to die and kill window, leaving zombie process.
Bug: 25089
Signed-off-by: Amadeusz Sławiński
---
src/screen.c | 2 +-
1 fi
