p910nd: Fix init script and make use use append_bool() and appand_string() functions
authorOliver Ertl <oliver@ertl-net.net>
Sun, 7 Oct 2007 18:32:58 +0000 (18:32 +0000)
committerOliver Ertl <oliver@ertl-net.net>
Sun, 7 Oct 2007 18:32:58 +0000 (18:32 +0000)
SVN-Revision: 9179

net/p910nd/files/p910nd.conf
net/p910nd/files/p910nd.init

index a9b61b2..8d6a04d 100644 (file)
@@ -1,4 +1,5 @@
 config p910nd
        option device        "/dev/usb/lp0"
-       option port           0
-       option bidirectional  1
+       option port          0
+       option bidirectional 1
+       option enabled       0
index bc21b20..1850c66 100644 (file)
@@ -2,21 +2,33 @@
 # Copyright (C) 2007 OpenWrt.org
 START=50
 
-BIN=p910nd
+append_bool() {
+       local section="$1"
+       local option="$2"
+       local value="$3"
+       local _val
+       config_get_bool _val "$section" "$option" '0'
+       [ "$_val" -gt 0 ] && append args "$3"
+}
 
-start_service() {
+append_string() {
        local section="$1"
-       config_get      port          "$section" port
-       config_get      device        "$section" device
-       config_get_bool bidirectional "$section" bidirectional
+       local option="$2"
+       local value="$3"
+       local _val
+       config_get _val "$section" "$option"
+       [ -n "$_val" ] && append args "$3$_val"
+}
 
-       options=""
-       [ $bidirectional -eq 1 ] && {
-               options="$options -b"
-       }
-       options="$options -f $device $port"
+start_service() {
+       local section="$1"
+       args=""
 
-       $BIN $options
+       append_bool "$section" bidirectional "-b"
+       append_string "$section" device "-f "
+       append_string "$section" port ""
+       config_get_bool "enabled" "$section" "enabled" '1'
+       [ "$enabled" -gt 0 ] && /usr/sbin/p910nd $args
 }
 
 stop_service() {