Package: kolab-cyrus-imapd
Version: 2.2.13p1-0.4
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 kolab-cyrus-imapd-2.2.13p1/debian/changelog kolab-cyrus-imapd-2.2.13p1/debian/changelog
--- kolab-cyrus-imapd-2.2.13p1/debian/changelog
+++ kolab-cyrus-imapd-2.2.13p1/debian/changelog
@@ -1,3 +1,10 @@
+kolab-cyrus-imapd (2.2.13p1-0.5) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fixe handling of /var/run/cyrus/socket folder life cycle (Closes: #XXXXXX).
+
+ -- Thomas Goirand <z...@debian.org>  Sat, 06 Oct 2012 18:34:45 +0800
+
 kolab-cyrus-imapd (2.2.13p1-0.4) unstable; urgency=low
 
   * Non-maintainer upload.
diff -u kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.dirs kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.dirs
--- kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.dirs
+++ kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.dirs
@@ -11 +10,0 @@
-var/run/cyrus/socket
diff -u kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.postinst kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.postinst
--- kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.postinst
+++ kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.postinst
@@ -53,11 +53,6 @@
 		dpkg-statoverride --update --add cyrus mail 755 /var/spool/cyrus
 	dpkg-statoverride --list /var/spool/sieve 2>&1 >/dev/null ||
 		dpkg-statoverride --update --add cyrus mail 755 /var/spool/sieve
-	dpkg-statoverride --list /var/run/cyrus 2>&1 >/dev/null ||
-		dpkg-statoverride --update --add cyrus mail 755 /var/run/cyrus
-	dpkg-statoverride --list /var/run/cyrus/socket 2>&1 >/dev/null ||
-		dpkg-statoverride --update --add cyrus mail 750 \
-				 /var/run/cyrus/socket
 
 	# Add user cyrus to group SASL, if such group exists
 	adduser cyrus sasl || true
diff -u kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.lintian kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.lintian
--- kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.lintian
+++ kolab-cyrus-imapd-2.2.13p1/debian/kolab-cyrus-common.lintian
@@ -3,2 +2,0 @@
-kolab-cyrus-common: dir-or-file-in-var-run var/run/cyrus/
-kolab-cyrus-common: dir-or-file-in-var-run var/run/cyrus/socket/

Reply via email to