redhat initscript: move restart block to a function

This ensures the "condrestart" option will work.

diff --git a/contrib/redhat/init.d-collectd b/contrib/redhat/init.d-collectd
index 4c69f3c9b7e0dd88fc1ac793eca7a7e68b02bb62..829eda5fcdc4d3c553f3c38d8c7a075aa4149c9d 100644 (file)
--- a/contrib/redhat/init.d-collectd
+++ b/contrib/redhat/init.d-collectd
@@ -42,7 +42,6 @@ check_config() {
         return 0
 }
 
-
 start () {
        echo -n $"Starting collectd: "
        check_config
@@ -58,6 +57,7 @@ start () {
                [ $RETVAL -eq 0 ] && touch /var/lock/subsys/$service
        fi
 }
+
 stop () {
        echo -n $"Stopping collectd: "
        killproc -p $COLLECTDMONPID -d $MAXWAIT $prog
@@ -65,6 +65,20 @@ stop () {
        echo
        [ $RETVAL -eq 0 ] && rm -f /var/lock/subsys/$service
 }
+
+restart () {
+       check_config
+       rc="$?"
+       if test "$rc" -ne 0; then
+               RETVAL=6
+               echo $"not restarting due to configuration error"
+               failure $"not restarting $service due to configuration error"
+       else
+               stop
+               start
+       fi
+}
+
 # See how we were called.
 case "$1" in
   start)
@@ -77,16 +91,7 @@ case "$1" in
        status -p $COLLECTDMONPID $prog
        ;;
   restart|reload)
-       check_config
-       rc="$?"
-       if test "$rc" -ne 0; then
-               RETVAL=6
-               echo $"not restarting due to configuration error"
-               failure $"not restarting $service due to configuration error"
-       else
-               stop
-               start
-       fi
+       restart
        ;;
   condrestart)
        [ -f /var/lock/subsys/$service ] && restart || :