Package: lmbench
Version: 3.0-a9-1
Severity: serious
Tags: patch

Dear Maintainer,

Andreas Beckmann <deb...@abeckmann.de> 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 lmbench-3.0-a9/debian/changelog lmbench-3.0-a9/debian/changelog
--- lmbench-3.0-a9/debian/changelog
+++ lmbench-3.0-a9/debian/changelog
@@ -1,3 +1,10 @@
+lmbench (3.0-a9-1.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fixes /var/run/lmbench life cycle handling (Closes: #XXXXXX).
+
+ -- Thomas Goirand <z...@debian.org>  Sun, 07 Oct 2012 13:57:23 +0000
+
 lmbench (3.0-a9-1) unstable; urgency=low
 
   * Update to latest upstream (3.0-a9).
diff -u lmbench-3.0-a9/debian/lmbench-run lmbench-3.0-a9/debian/lmbench-run
--- lmbench-3.0-a9/debian/lmbench-run
+++ lmbench-3.0-a9/debian/lmbench-run
@@ -12,6 +12,8 @@
 export SHAREDIR DATADIR BINDIR SCRIPTSDIR RESULTSDIR SRCDIR
 CONFIG=$DATADIR/config/`$SCRIPTSDIR/config`
 
+mkdir -p /var/run/lmbench
+
 runuid=`id -u`
 
 [ $runuid -gt 0 ] && {
diff -u lmbench-3.0-a9/debian/lmbench.dirs lmbench-3.0-a9/debian/lmbench.dirs
--- lmbench-3.0-a9/debian/lmbench.dirs
+++ lmbench-3.0-a9/debian/lmbench.dirs
@@ -15,3 +15,2 @@
 var/lib/lmbench/results
-var/run/lmbench/
 var/tmp/lmbench/
only in patch2:
unchanged:
--- lmbench-3.0-a9.orig/debian/lmbench.postrm
+++ lmbench-3.0-a9/debian/lmbench.postrm
@@ -0,0 +1,7 @@
+#!/bin/sh
+
+set -e
+
+if [ "${1}" = "purge" ] ; then
+	rm -rf /var/run/lmbench
+fi

Reply via email to