Package: couriergraph Version: 0.25-4.1 Severity: serious Tags: patch Dear Maintainer,
Andreas Beckmann <[email protected]> reported in -devel that your package (as well as 27 others) ships a folder either in /var/run or /var/lock. This is forbidden by policy. Lintian detects the problem and warns as follow: /var/run may be a temporary filesystem, so any directories or files needed /there must be created dynamically at boot time. Refer to Debian Policy Manual section 9.3.2 (Writing the scripts) for details. Severity: serious, Certainty: possible Check: files, Type: binary, udeb which is why I am reporting this bug with severity serious (and there fore, release critical). Please fix your package. I have attached what I believe is a good fix the problem, however, I haven't tried it, and I haven't tested if something more for creating the necessary folder at runtime should be added. Please make sure to test before applying the patch blindly. Cheers, Thomas Goirand (zigo)
diff -u couriergraph-0.25/debian/changelog couriergraph-0.25/debian/changelog --- couriergraph-0.25/debian/changelog +++ couriergraph-0.25/debian/changelog @@ -1,3 +1,10 @@ +couriergraph (0.25-4.2) unstable; urgency=low + + * Non-maintainer upload. + * Doesn't ship /var/run/servergraph in the .deb anymore (Closes: #XXXXXX). + + -- Thomas Goirand <[email protected]> Sat, 06 Oct 2012 16:18:50 +0800 + couriergraph (0.25-4.1) unstable; urgency=low * Non-maintainer upload. diff -u couriergraph-0.25/debian/dirs couriergraph-0.25/debian/dirs --- couriergraph-0.25/debian/dirs +++ couriergraph-0.25/debian/dirs @@ -4 +3,0 @@ -var/run/servergraph diff -u couriergraph-0.25/debian/init couriergraph-0.25/debian/init --- couriergraph-0.25/debian/init +++ couriergraph-0.25/debian/init @@ -23,6 +23,9 @@ if [ -f $CONFIG ]; then . $CONFIG fi + +mkdir -p /var/run/servergraph || true +chown daemon:root /var/run/servergraph || true case "$1" in start) diff -u couriergraph-0.25/debian/postinst couriergraph-0.25/debian/postinst --- couriergraph-0.25/debian/postinst +++ couriergraph-0.25/debian/postinst @@ -27,7 +27,6 @@ # installation fails and the `postinst' is called with `abort-upgrade', # `abort-remove' or `abort-deconfigure'. -RUNDIR=/var/run/servergraph DATADIR=/var/lib/couriergraph CACHEDIR=/var/cache/couriergraph DEFAULTS=/etc/default/couriergraph @@ -35,9 +34,6 @@ case "$1" in configure) - mkdir -p $RUNDIR || true - chown daemon:root $RUNDIR - if [ ! -d $DATADIR ]; then mkdir $DATADIR; fi chown root:adm $DATADIR chmod 775 $DATADIR
