Use exec when starting process via the shell.

This fixes the signal handling issue with systemctl reported in jamulussoftware#1515, but still allows shell substitution of environment variables.
softins · Apr 14, 2021 · 94e31bb · 94e31bb
1 parent 3df60d6
commit 94e31bb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/distributions/debian/jamulus-headless.service b/distributions/debian/jamulus-headless.service
@@ -16,7 +16,7 @@ IOSchedulingPriority=0
 
 #### Change this to publish this server, set genre, location and other parameters.
 #### See https://jamulus.io/wiki/Command-Line-Options ####
-ExecStart=/bin/sh -c '/usr/bin/jamulus-headless -s -n'
+ExecStart=/bin/sh -c 'exec /usr/bin/jamulus-headless -s -n'
 
 
 Restart=on-failure

diff --git a/distributions/jamulus-server.service b/distributions/jamulus-server.service
@@ -8,7 +8,7 @@ Type=simple
 Restart=always
 RestartSec=1
 User=jamulus
-ExecStart=/bin/sh -c '/usr/bin/jamulus -s -n -l /var/log/jamulus -e jamulus.fischvolk.de -g -o "$(uname -n);;"'
+ExecStart=/bin/sh -c 'exec /usr/bin/jamulus -s -n -l /var/log/jamulus -e jamulus.fischvolk.de -g -o "$(uname -n);;"'
 
 [Install]
 WantedBy=multi-user.target