[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#27553: [PATCH shepherd] Register SIGCHLD handler after primitive for
|
From: |
Ludovic Courtès |
|
Subject: |
bug#27553: [PATCH shepherd] Register SIGCHLD handler after primitive fork |
|
Date: |
Thu, 07 Sep 2017 16:49:36 +0200 |
|
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.2 (gnu/linux) |
Heya!
Jelle Licht <address@hidden> skribis:
> I just tested what Ludo' proposed, and it seems to work like a charm.
> Seeing as we might be seeing more non-init shepherd instances w.r.t.
> user services and the possible service extension to `guix environment',
> I think it would be a good call to fix this bug :-).
Indeed, thanks for the reminder.
> From db942182224dfc0accad94897dd2122b128eef07 Mon Sep 17 00:00:00 2001
> From: Jelle Licht <address@hidden>
> Date: Thu, 7 Sep 2017 00:52:49 +0200
> Subject: [PATCH] Lazily register SIGCHLD hander on first call to
> 'fork+exec-command'.
>
> * modules/shepherd.scm (main): Move unconditional top-level call to
> 'sigaction' to...
> * modules/shepherd/service.scm (fork+exec-command): here. Use new variable.
> (%sigchld-handler-installed?): New variable.
LGTM, applied!
Ludo’.