branch master updated: services: pcscd: Fix daemon signal handling.

[guix-commits]

This is an automated email from the git hooks/post-receive script.

bricewge pushed a commit to branch master
in repository guix.

The following commit(s) were added to refs/heads/master by this push:
     new e789ce5  services: pcscd: Fix daemon signal handling.
e789ce5 is described below

commit e789ce538ed848bacb8f4eb5742f78b965ccf57c
Author: Brice Waegeneire <brice@waegenei.re>
AuthorDate: Tue Jun 15 23:24:30 2021 +0200

    services: pcscd: Fix daemon signal handling.
    
    "pcscd" wouldn't handle SIGTERM as it inherit ignoring this signal (and
    others) from its parent shepherd; fork+exec-command restore signal
    handling.  Fixes <https://issues.guix.gnu.org/45202>.
    
    * gnu/services/security-token.scm (pcscd)[start]: Use
    fork+exec-command to start "pcscd".
---
 gnu/services/security-token.scm | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gnu/services/security-token.scm b/gnu/services/security-token.scm
index 0cbb591..52afad8 100644
--- a/gnu/services/security-token.scm
+++ b/gnu/services/security-token.scm
@@ -61,8 +61,10 @@
                    (let ((socket "/run/pcscd/pcscd.comm"))
                      (when (file-exists? socket)
                        (delete-file socket)))
-                   (invoke #$(file-append pcsc-lite "/sbin/pcscd"))
-                   (call-with-input-file "/run/pcscd/pcscd.pid" read)))
+                   (fork+exec-command
+                    (list #$(file-append pcsc-lite "/sbin/pcscd")
+                          "--foreground")
+                    #:log-file "/var/log/pcscd.log")))
         (stop #~(make-kill-destructor)))))))
 
 (define pcscd-activation