Initd fixes (#10)

* stop service when uninstalling package

* Use initd daemon func instead of nohup

If a user were to try to install bashrpc over ssh
where there is not a tty shell, then nohup is
completely ineffective.

Author: binarymason

dist/init/sysvinit/bashrpc

@@ -12,20 +12,22 @@
 DESC="bashrpc"
 NAME=bashrpc
 DAEMON=/usr/local/bin/bashrpc
-
-
+PIDFILE=/var/run/$NAME.pid
 LOGFILE=/var/log/$NAME.log
 CONFIGFILE=/etc/bashrpc/bashrpc.yml
 
 test -x $DAEMON || exit 0
 
+# source init.d functions
+. /etc/init.d/functions
+
 start() {
     touch $LOGFILE
-    nohup "$DAEMON" -c "$CONFIGFILE" --log "$LOGFILE" &
+    daemon --pidfile "$PIDFILE" "$DAEMON" -c "$CONFIGFILE" --log "$LOGFILE"
 }
 
 stop() {
-  pkill -f "$DAEMON"
+  killproc -p "$PIDFILE"
 }
 
 reload() {

dist/scripts/postremove.sh

@@ -2,6 +2,9 @@
 
 service_manager=$(ps --no-headers -o comm 1)
 
+# stop the bashrpc service if it's running
+service bashrpc stop &>/dev/null
+
 if [ "$service_manager" = init ]; then
   echo "+ removing init script"
   rm -f /etc/init.d/bashrpc