Rewrite handling of /etc/default/iodine
authorgregor herrmann <gregoa@debian.org>
Thu, 9 May 2013 12:54:49 +0000 (14:54 +0200)
committergregor herrmann <gregoa@debian.org>
Thu, 9 May 2013 12:54:49 +0000 (14:54 +0200)
according to debconf-devel(7) in debian/config and debian/postinst.

debian/config
debian/postinst

index c7f05444da9fbdd5ac31e064055e9273a15ceca5..020a21470956ff6b1f54b82b3f5cb65d9fd917f7 100644 (file)
@@ -1,30 +1,18 @@
 #!/bin/sh
 
+CONFIGFILE=/etc/default/iodine
 set -e
-
 . /usr/share/debconf/confmodule
 
-getvalue() {
-       awk '/^'"$1"'=/ {sub("^'"$1"'=\"", ""); sub("\"$", ""); value=$0;} END {print value}' /etc/default/iodine
-}
-
-if [ -s /etc/default/iodine ] ; then
-       START_IODINED="$(getvalue START_IODINED)"
-       if [ -n "$START_IODINED" ] ; then
-               db_set iodine/start_daemon "$START_IODINED"
-       fi
-       IODINED_ARGS="$(getvalue IODINED_ARGS)"
-       if [ -n "$IODINED_ARGS" ] ; then
-               db_set iodine/daemon_options "$IODINED_ARGS"
-       fi
-       IODINED_PASSWORD="$(getvalue IODINED_PASSWORD)"
-       if [ -n "$IODINED_PASSWORD" ] ; then
-               db_set iodine/daemon_password "$IODINED_PASSWORD"
-       fi
+if [ -s $CONFIGFILE ] ; then
+       . $CONFIGFILE
+       db_set iodine/start_daemon    "$START_IODINED"
+       db_set iodine/daemon_options  "$IODINED_ARGS"
+       db_set iodine/daemon_password "$IODINED_PASSWORD"
 fi
 
 db_input medium iodine/start_daemon || true
-db_go
+db_go || true
 
 db_get iodine/start_daemon
 START_DAEMON=$RET
@@ -32,5 +20,5 @@ START_DAEMON=$RET
 if [ "x$START_DAEMON" = "xtrue" ] ; then
        db_input medium iodine/daemon_options || true
        db_input medium iodine/daemon_password || true
-       db_go
+       db_go || true
 fi
index 80498c045b65d49d591474c33dad6f39f6188211..c6649470d7c474f7ee73338af22b3491b44e1d0e 100644 (file)
@@ -3,8 +3,8 @@
 #
 # see: dh_installdeb(1)
 
+CONFIGFILE=/etc/default/iodine
 set -e
-
 . /usr/share/debconf/confmodule
 
 # summary of how this script can be called:
@@ -28,23 +28,46 @@ case "$1" in
             cd /dev
             ./MAKEDEV tun || true
         fi 
+
         # and we want a special user
         adduser --quiet --system --home /var/run/iodine iodine
-        # generate /etc/default/iodine
+
+        # generate/update /etc/default/iodine
+        if [ ! -e $CONFIGFILE ]; then
+            cat <<EOF >$CONFIGFILE
+# Default settings for iodine. This file is sourced from
+# /etc/init.d/iodined
+START_IODINED=
+IODINED_ARGS=
+IODINED_PASSWORD=
+EOF
+        fi
+
         db_get iodine/start_daemon
         START_IODINED=$RET
         db_get iodine/daemon_options
         IODINED_ARGS=$RET
         db_get iodine/daemon_password
         IODINED_PASSWORD=$RET
-        cat <<EOF > /etc/default/iodine
-# Default settings for iodine. This file is sourced from
-# /etc/init.d/iodined
-START_IODINED="$START_IODINED"
-IODINED_ARGS="$IODINED_ARGS"
-IODINED_PASSWORD="$IODINED_PASSWORD"
-EOF
-        [ -e /etc/default/iodine ] && chmod 600 /etc/default/iodine
+
+        cp -a -f $CONFIGFILE $CONFIGFILE.tmp
+
+        # If the admin deleted or commented some variables but then set
+        # them via debconf, (re-)add them to the conffile.
+        test -z "$START_IODINED" || grep -Eq '^ *START_IODINED=' $CONFIGFILE || \
+            echo "START_IODINED=" >> $CONFIGFILE
+        test -z "$IODINED_ARGS" || grep -Eq '^ *IODINED_ARGS=' $CONFIGFILE || \
+            echo "IODINED_ARGS=" >> $CONFIGFILE
+        test -z "$IODINED_PASSWORD" || grep -Eq '^ *IODINED_PASSWORD=' $CONFIGFILE || \
+            echo "IODINED_PASSWORD=" >> $CONFIGFILE
+
+        sed -e "s/^ *START_IODINED=.*/START_IODINED=\"$START_IODINED\"/" \
+            -e "s/^ *IODINED_ARGS=.*/IODINED_ARGS=\"$IODINED_ARGS\"/" \
+            -e "s/^ *IODINED_PASSWORD=.*/IODINED_PASSWORD=\"$IODINED_PASSWORD\"/" \
+            < $CONFIGFILE > $CONFIGFILE.tmp
+
+        mv -f $CONFIGFILE.tmp $CONFIGFILE
+        [ ! -e $CONFIGFILE ] || chmod 600 $CONFIGFILE
     ;;
 
     abort-upgrade|abort-remove|abort-deconfigure)