dhcp: adjust initscripts and misc details

Revamp initscripts see bug #3601
* Trim whitespace
* Check for installed binary (dhcp-relay)
* Check for installed config (dhcp-server)
* Check for proper variable setup from the script (both)
* Add "FAIL" message when startup fails (blind OK isn't accurate)
* Add start/stop messages to dhcp-relay

Switch bool to "dhcp (ISC)" to keep in line with package sorting.

Signed-off-by: Gustavo Zacarias <gustavo@zacarias.com.ar>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>

diff --git a/package/dhcp/Config.in b/package/dhcp/Config.in
index e341485b1627cf22cf77782516aecfb8959bc571..7944edefc9c5f5bbdccd482499729de60b4a5f80 100644 (file)
--- a/package/dhcp/Config.in
+++ b/package/dhcp/Config.in
@@ -1,5 +1,5 @@
 config BR2_PACKAGE_DHCP
-       bool "isc dhcp"
+       bool "dhcp (ISC)"
        # fork()
        depends on BR2_USE_MMU
        help

diff --git a/package/dhcp/S80dhcp-relay b/package/dhcp/S80dhcp-relay
index c870ea7f5d63b66a93e13c3f65e567879642347a..2a632ef95083ff02bcde1aa7db67589c45c61d49 100755 (executable)
--- a/package/dhcp/S80dhcp-relay
+++ b/package/dhcp/S80dhcp-relay
@@ -3,7 +3,6 @@
 # $Id: dhcp3-relay,v 1.1 2004/04/16 15:41:08 ml Exp $
 #
 
-
 # What servers should the DHCP relay forward requests to?
 # e.g: SERVERS="192.168.0.1"
 SERVERS=""
@@ -14,31 +13,29 @@ INTERFACES=""
 # Additional options that are passed to the DHCP relay daemon?
 OPTIONS=""
 
-
-
-
-# It is not safe to start if we don't have a default configuration...
-echo "/etc/init.d/dhcp-relay not yet configured! - Aborting..."
-exit 1;
-
-
+# Sanity checks
+test -f /usr/sbin/dhcrelay || exit 0
+test -n "$INTERFACES" || exit 0
+test -n "$SERVERS" || exit 0
 
 # Build command line for interfaces (will be passed to dhrelay below.)
 IFCMD=""
-if test "$INTERFACES" != ""; then
-       for I in $INTERFACES; do
-               IFCMD=${IFCMD}"-i "${I}" "
-       done
-fi
+for I in $INTERFACES; do
+       IFCMD=${IFCMD}"-i "${I}" "
+done
 
 DHCRELAYPID=/var/run/dhcrelay.pid
 
 case "$1" in
        start)
+               echo -n "Starting DHCP relay: "
                start-stop-daemon -S -x /usr/sbin/dhcrelay -- -q $OPTIONS $IFCMD $SERVERS
+               [ $? == 0 ] && echo "OK" || echo "FAIL"
                ;;
        stop)
+               echo -n "Stopping DHCP relay: "
                start-stop-daemon -K -x /usr/sbin/dhcrelay
+               [ $? == 0 ] && echo "OK" || echo "FAIL"
                ;;
        restart | force-reload)
                $0 stop
@@ -47,7 +44,7 @@ case "$1" in
                ;;
        *)
                echo "Usage: /etc/init.d/dhcp-relay {start|stop|restart|force-reload}"
-               exit 1 
+               exit 1
 esac
 
 exit 0

diff --git a/package/dhcp/S80dhcp-server b/package/dhcp/S80dhcp-server
index 7c7f893a149a12ff9073b4d1cba90ea4b1b8e93b..98bd3cc67e42b98b0904fb64da7ee2d81d14f6ed 100755 (executable)
--- a/package/dhcp/S80dhcp-server
+++ b/package/dhcp/S80dhcp-server
@@ -3,34 +3,27 @@
 # $Id: dhcp3-server.init.d,v 1.4 2003/07/13 19:12:41 mdz Exp $
 #
 
-
-
 # On what interfaces should the DHCP server (dhcpd) serve DHCP requests?
 #       Separate multiple interfaces with spaces, e.g. "eth0 eth1".
 INTERFACES=""
 
-
-# It is not safe to start if we don't have a default configuration...
-echo "/etc/init.d/dhcp-server not yet configured! - Aborting..."
-exit 1;
-
-
-
+# Sanity checks
 test -f /usr/sbin/dhcpd || exit 0
-
+test -f /etc/dhcpd.conf || exit 0
+test -n "$INTERFACES" || exit 0
 
 case "$1" in
        start)
                echo -n "Starting DHCP server: "
                test -d /var/lib/dhcp/ || mkdir -p /var/lib/dhcp/
-               test -f /var/lib/dhcp/dhcpd.leases || touch /var/lib/dhcp/dhcpd.leases  
+               test -f /var/lib/dhcp/dhcpd.leases || touch /var/lib/dhcp/dhcpd.leases
                start-stop-daemon -S -x /usr/sbin/dhcpd -- -q $INTERFACES
-               echo "."
+               [ $? == 0 ] && echo "OK" || echo "FAIL"
                ;;
        stop)
                echo -n "Stopping DHCP server: dhcpd3"
                start-stop-daemon -K -x /usr/sbin/dhcpd
-               echo "."
+               [ $? == 0 ] && echo "OK" || echo "FAIL"
                ;;
        restart | force-reload)
                $0 stop
@@ -42,7 +35,7 @@ case "$1" in
                ;;
        *)
                echo "Usage: /etc/init.d/dhcp-server {start|stop|restart|force-reload}"
-               exit 1 
+               exit 1
 esac
 
 exit 0