Bug#856822: iodine: /usr/lib/tmpfiles.d/iodined.conf should use /run not /var/run

2017-03-10 Thread gregor herrmann
Control: tag -1 + confirmed

On Sun, 05 Mar 2017 15:52:11 +1100, Russell Coker wrote:

> As /var/run has been a symlink to /run for a long time configuration should
> point to /run.  /usr/lib/tmpfiles.d/iodined.conf should create /run/iodine
> not /var/run/iodine .

Nice catch, thank you.

It looks like there are more files affected in practice:

% grep -r /var/run
debian/changelog: [..]
debian/iodine.iodined.init:PIDFILE=/var/run/$NAME.pid
debian/iodine.iodined.init:CHROOTDIR=/var/run/iodine
debian/postinst:adduser --quiet --system --home /var/run/iodine iodine
debian/iodined.tmpfile:d /var/run/iodine - - -
debian/iodined.service:ExecStart=/usr/sbin/iodined -f -u iodine -t 
/var/run/iodine $IODINED_ARGS -P ${IODINED_PASSWORD}

This looks all quite straight forward, except maybe for the home
directory of the iodine user. But I suppose a usermod in the preinst
should do the trick here.

I'll try to get this implemented for the first upload after the
stretch release.


Cheers,
gregor

-- 
 .''`.  https://info.comodo.priv.at/ - Debian Developer https://www.debian.org
 : :' : OpenPGP fingerprint D1E1 316E 93A7 60A8 104D  85FA BB3A 6801 8649 AA06
 `. `'  Member of VIBE!AT & SPI, fellow of the Free Software Foundation Europe
   `-   NP: Kurti Ostbahn


signature.asc
Description: Digital Signature


Bug#856822: iodine: /usr/lib/tmpfiles.d/iodined.conf should use /run not /var/run

2017-03-04 Thread Russell Coker
Package: iodine
Version: 0.7.0-7
Severity: normal

As /var/run has been a symlink to /run for a long time configuration should
point to /run.  /usr/lib/tmpfiles.d/iodined.conf should create /run/iodine
not /var/run/iodine .