Use exec in order to get rid of bash processes (#439)

GioF71 · Sep 26, 2024 · f138106 · f138106
1 parent 50adfa1
commit f138106
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/app/bin/run-upmpdcli.sh b/app/bin/run-upmpdcli.sh
@@ -885,8 +885,8 @@ echo "CMD_LINE=[${CMD_LINE}]"
 
 if [[ $current_user_id -eq 0 ]]; then
     echo "USER MODE [$USER_NAME]"
-    su - $USER_NAME -c "$CMD_LINE"
+    exec su - $USER_NAME -c "$CMD_LINE"
 else
     echo "Running as current uid [$current_user_id] ..."
-    eval "$CMD_LINE"
+    eval "exec $CMD_LINE"
 fi
diff --git a/doc/change-history.md b/doc/change-history.md
@@ -2,6 +2,7 @@
 
 Change Date|Major Changes
 ---|---
+2024-09-26|Use exec in order to get rid of bash processes
 2024-08-24|Fix arm-only image build issue (see issue [#436](https://github.com/GioF71/upmpdcli-docker/issues/436))
 2024-08-21|Bump to upmpdcli version 1.8.6
 2024-07-19|Add `SKIP_CHOWN_CACHE` to skip chown on possibly crowded /cache