Package: chef-solr
Version: 10.12.0+dfsg-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 -Nru chef-solr-10.12.0+dfsg/debian/changelog chef-solr-10.12.0+dfsg/debian/changelog
--- chef-solr-10.12.0+dfsg/debian/changelog	2012-06-27 06:20:11.000000000 +0000
+++ chef-solr-10.12.0+dfsg/debian/changelog	2012-10-07 15:24:14.000000000 +0000
@@ -1,3 +1,10 @@
+chef-solr (10.12.0+dfsg-1.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fixes handling of /var/run/chef life cycle (Closes: #XXXXXX).
+
+ -- Thomas Goirand <[email protected]>  Sun, 07 Oct 2012 15:23:49 +0000
+
 chef-solr (10.12.0+dfsg-1) unstable; urgency=low
 
   * New upstream version.
diff -Nru chef-solr-10.12.0+dfsg/debian/chef-solr.dirs chef-solr-10.12.0+dfsg/debian/chef-solr.dirs
--- chef-solr-10.12.0+dfsg/debian/chef-solr.dirs	2012-06-20 19:20:40.000000000 +0000
+++ chef-solr-10.12.0+dfsg/debian/chef-solr.dirs	2012-10-07 15:22:18.000000000 +0000
@@ -7,7 +7,6 @@
 var/lib/chef/solr/solr-jetty/solr/WEB-INF
 var/cache/chef/solr/data
 var/log/chef
-var/run/chef
 etc/chef
 etc/chef/solr-jetty
 usr/sbin
diff -Nru chef-solr-10.12.0+dfsg/debian/chef-solr.postinst chef-solr-10.12.0+dfsg/debian/chef-solr.postinst
--- chef-solr-10.12.0+dfsg/debian/chef-solr.postinst	2012-06-20 19:20:40.000000000 +0000
+++ chef-solr-10.12.0+dfsg/debian/chef-solr.postinst	2012-10-07 15:25:35.000000000 +0000
@@ -57,7 +57,6 @@
     chown -R $USER:$GROUP /var/lib/chef
     chown -R $USER:$GROUP /var/log/chef
     chown -R $USER:$GROUP /var/cache/chef
-    chown -R $USER:$GROUP /var/run/chef
   ;;
 
   abort-upgrade|abort-remove|abort-deconfigure)
diff -Nru chef-solr-10.12.0+dfsg/debian/chef-solr.postrm chef-solr-10.12.0+dfsg/debian/chef-solr.postrm
--- chef-solr-10.12.0+dfsg/debian/chef-solr.postrm	2012-06-20 19:20:40.000000000 +0000
+++ chef-solr-10.12.0+dfsg/debian/chef-solr.postrm	2012-10-07 15:23:35.000000000 +0000
@@ -10,6 +10,7 @@
 
 case "$1" in
     purge)
+      rm -rf /var/run/chef
       rm -rf /var/log/chef/solr*
       test -x /usr/bin/ucf && ucf --purge $TEMPLATEFILE $CONFIGFILE
       rm -rf $CONFIGFILE $CONFIGFILE.ucf-old $CONFIGFILE.ucf-new $CONFIGFILE.ucf-dist

Reply via email to