From 40c35604536a23783fd492328e64b86a2afffa37 Mon Sep 17 00:00:00 2001
From: Daniel O'Connor <darius@dons.net.au>
Date: Wed, 5 Jul 2017 10:40:10 +0930
Subject: [PATCH] Change simplify and update FreeBSD init script.

Use daemon and PID file functionality.
Send signal to kill as it shuts down cleanly.
Remove web shutdown as it is redundant and doesn't work with SSL.
---
 init-scripts/init.freebsd | 55 +++------------------------------------
 1 file changed, 4 insertions(+), 51 deletions(-)

diff --git a/init-scripts/init.freebsd b/init-scripts/init.freebsd
index 2a2374ac..9d967844 100755
--- a/init-scripts/init.freebsd
+++ b/init-scripts/init.freebsd
@@ -19,18 +19,10 @@
 #     Default is same as sickgear_dir.
 # sickgear_datadir:  Data directory for Sick Beard (DB, Logs, config)
 #     Default is same as sickgear_chdir
-# sickgear_pid:  The name of the pidfile to create.
+# sickgear_pidfile:  The name of the pidfile to create.
 #     Default is sickgear.pid in sickgear_dir.
-# sickgear_host: The hostname or IP SickGear is listening on
-#     Default is 127.0.0.1
 # sickgear_port: The port SickGear is listening on
 #     Default is 8081
-# sickgear_web_user: Username to authenticate to the SickGear web interface
-#     Default is an empty string (no username)
-# sickgear_web_password: Password to authenticate to the SickGear web interface
-#     Default is an empty string (no password)
-# sickgear_webroot: Set to value of web_root in config (for proxies etc)
-#     Default is an empty string (if set must start with a "/")
 PATH="/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/sbin:/usr/local/bin"
 
 . /etc/rc.subr
@@ -45,54 +37,15 @@ load_rc_config ${name}
 : ${sickgear_dir:="/usr/local/sickgear"}
 : ${sickgear_chdir:="${sickgear_dir}"}
 : ${sickgear_datadir:="${sickgear_chdir}"}
-: ${sickgear_pid:="${sickgear_dir}/sickgear.pid"}
+: ${sickgear_pidfile:="${sickgear_dir}/sickgear.pid"}
 : ${sickgear_host:="127.0.0.1"}
 : ${sickgear_port:="8081"}
-: ${sickgear_web_user:=""}
-: ${sickgear_web_password:=""}
-: ${sickgear_webroot:=""}
 
-status_cmd="${name}_status"
-stop_cmd="${name}_stop"
-
-command="/usr/sbin/daemon"
-command_args="-f -p ${sickgear_pid} python ${sickgear_dir}/SickBeard.py --quiet --nolaunch"
+command="/usr/local/bin/python"
+command_args="${sickgear_dir}/SickBeard.py --daemon --pidfile ${sickgear_pidfile}"
 
 # Add datadir to the command if set
 [ ! -z "${sickgear_datadir}" ] && \
   command_args="${command_args} --datadir ${sickgear_datadir}"
 
-# Ensure user is root when running this script.
-if [ `id -u` != "0" ]; then
-  echo "Oops, you should be root before running this!"
-  exit 1
-fi
-
-verify_sickgear_pid() {
-    # Make sure the pid corresponds to the SickGear process.
-    pid=`cat ${sickgear_pid} 2>/dev/null`
-    ps -p ${pid} 2>/dev/null | grep -q "python ${sickgear_dir}/SickBeard.py"
-    return $?
-}
-
-# Try to stop SickGear cleanly by calling shutdown over http.
-sickgear_stop() {
-    echo "Stopping $name"
-    verify_sickgear_pid
-    sickgear_url="${sickgear_host}:${sickgear_port}"
-    [ ! -z "${sickgear_web_user}" ] && \
-       sickgear_url="${sickgear_web_user}:${sickgear_web_password}@${sickgear_url}"
-    [ ! -z "${sickgear_webroot}" ] && \
-       sickgear_url="${sickgear_url}${sickgear_webroot}"
-    fetch -o - -q "http://${sickgear_url}/home/shutdown/?pid=${pid}" >/dev/null
-    if [ -n "${pid}" ]; then
-      wait_for_pids ${pid}
-      echo "Stopped"
-    fi
-}
-
-sickgear_status() {
-    verify_sickgear_pid && echo "$name is running as ${pid}" || echo "$name is not running"
-}
-
 run_rc_command "$1"