Package: iodine
Version: 0.6.0~rc1-4
Severity: serious

When I upgrade iodine it'll overwrite my changes in
/etc/default/iodine. This appears to be done by this bit in
debian/postinst:

            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

Since it sources the file it'll be changed like this after upgrade:

    $ sudo git diff default/iodine
    diff --git a/default/iodine b/default/iodine
    index 4132933..b9384c4 100644
    --- a/default/iodine
    +++ b/default/iodine
    @@ -2,4 +2,4 @@
     # /etc/init.d/iodined
     START_IODINED="true"
     IODINED_ARGS="-c -p 5252 172.31.255.1/27 MY-HOSTNAME"
    -IODINED_PASSWORD="$(cat /etc/iodine/passwd/iodine)"
    +IODINED_PASSWORD="my seekrt password"

Where "my seekrt password" is the contents of
/etc/iodine/passwd/iodine. There's two things wrong with this:

 * /etc/default/iodine is sourced as a shell file, and due to
   constructs like $() you can't source it and write out the relevant
   variables again without making changes like these.

 * /etc/default/iodine is a configuration file. You should ask before
   modifying it as most Debian packages do (per policy) for their
   configuration files. The iodine package isn't doing this, hence
   marking this bug as "serious".
    
-- System Information:
Debian Release: 6.0
  APT prefers testing
  APT policy: (900, 'testing'), (800, 'unstable'), (700, 'stable'), (1, 
'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_US.utf8, LC_CTYPE=en_US.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages iodine depends on:
ii  adduser                 3.112+nmu2       add and remove users and groups
ii  debconf [debconf-2.0]   1.5.36.1         Debian configuration management sy
ii  libc6                   2.11.2-10        Embedded GNU C Library: Shared lib
ii  makedev                 2.3.1-89         creates device files in /dev
ii  udev                    164-3            /dev/ and hotplug management daemo
ii  zlib1g                  1:1.2.3.4.dfsg-3 compression library - runtime

iodine recommends no packages.

Versions of packages iodine suggests:
ii  dnsutils             1:9.7.2.dfsg.P3-1.1 Clients provided with BIND
pn  fping                <none>              (no description available)
ii  gawk                 1:3.1.7.dfsg-5      GNU awk, a pattern scanning and pr
pn  ipcalc               <none>              (no description available)
ii  iproute              20100519-3          networking and traffic control too

-- debconf information excluded



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to