Your message dated Sun, 10 Aug 2014 06:03:35 +0000
with message-id <e1xgmeb-0007eu...@franck.debian.org>
and subject line Bug#689892: fixed in havp 0.92a-3
has caused the Debian Bug report #689892,
regarding Ships a folder in /var/run or /var/lock (Policy Manual section 9.3.2)
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
689892: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=689892
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: havp
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 havp-0.92a/debian/changelog havp-0.92a/debian/changelog
--- havp-0.92a/debian/changelog
+++ havp-0.92a/debian/changelog
@@ -1,3 +1,10 @@
+havp (0.92a-2.1) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * Fixes (Closes: #XXXXXX).
+
+ -- Thomas Goirand <z...@debian.org>  Sun, 07 Oct 2012 11:05:10 +0000
+
 havp (0.92a-2) unstable; urgency=low
 
   * Include debconf language updates from previous NMU and new updates.
diff -u havp-0.92a/debian/havp.init havp-0.92a/debian/havp.init
--- havp-0.92a/debian/havp.init
+++ havp-0.92a/debian/havp.init
@@ -37,6 +37,13 @@
 
 set -e
 
+if [ ! -d ${RUNDIR} ] ; then
+	mkdir -p ${RUNDIR}
+	if [ -d ${RUNDIR} ] ; then
+		chown ${USER}:${GROUP} ${RUNDIR}
+	fi
+fi
+
 case "$1" in
   start)
         # Create the run directory if it doesn't exist
diff -u havp-0.92a/debian/havp.postinst havp-0.92a/debian/havp.postinst
--- havp-0.92a/debian/havp.postinst
+++ havp-0.92a/debian/havp.postinst
@@ -37,7 +37,6 @@
       fi
       # care for proper ownership in any case (e.g. updating from an earlier 
       # havp package with different paths)
-      chown havp:havp /var/run/havp
       chown havp:havp /var/log/havp
       chown havp:havp /var/lib/havp
       chown havp:havp /var/spool/havp
diff -u havp-0.92a/debian/havp.postrm havp-0.92a/debian/havp.postrm
--- havp-0.92a/debian/havp.postrm
+++ havp-0.92a/debian/havp.postrm
@@ -29,7 +29,7 @@
           rm /var/lib/havp/havp.tmp
           rm /etc/default/havp
         fi
-
+	rm -rf /var/run/havp
         ;;
 
        remove|upgrade|failed-upgrade|abort-install|abort-upgrade|disappear)

--- End Message ---
--- Begin Message ---
Source: havp
Source-Version: 0.92a-3

We believe that the bug you reported is fixed in the latest version of
havp, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 689...@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Scott Kitterman <sc...@kitterman.com> (supplier of updated havp package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmas...@ftp-master.debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Sun, 10 Aug 2014 01:12:27 -0400
Source: havp
Binary: havp
Architecture: source i386
Version: 0.92a-3
Distribution: unstable
Urgency: medium
Maintainer: ClamAV Team <pkg-clamav-de...@lists.alioth.debian.org>
Changed-By: Scott Kitterman <sc...@kitterman.com>
Description:
 havp       - HTTP Anti Virus Proxy
Closes: 628612 628613 628615 664346 666016 689778 689892
Changes:
 havp (0.92a-3) unstable; urgency=medium
 .
   [ Sebastian Andrzej Siewior ]
   * convert package to git-dpm and drop dpatch (Closes: #664346).
   * Do not ship /var/run/havp, patch by Thomas Goirand (Closes: #689892).
   * purge /var/run/havp und removal (Closes: #689778).
   * remove the -x flag from .html files in the debian folder where the package
     got intalled to actually fix lintian executable-not-elf-or-script.
   * redo the rules with minimal debhelper 8
 .
   [ Andreas Cadhalpun ]
   * Use 'set -e' in maintainer scripts.
   * Add homepage to debian/control.
   * Add CPPFLAGS to CFLAGS, because CPPFLAGS are not respected by havp's
     build system.
   * Call 'modprobe loop' in the init script before trying to mount a loop
     device. This is necessary for newer kernels.
   * Remove all logs in /var/log/havp, the contents of /var/lib/havp and
     /etc/default/havp on purge (Closes: #666016).
   * Fix restart action of init script. (Closes: #628612)
     Thanks to Laurent Baysse for the bug report and patch.
 .
   [ Scott Kitterman ]
   * Agreed maintainer change to pkg-clamav team, added myself, Sebastian
     Andrzej Siewior, and Andreas Cadhalpun as uploaders along with Rene
     Mayrhofer (the previous maintainer)
   * Fix PID file naming for reload statement in havp.init, partially addresses
     #524045
   * Add lsb-base (>= 3.2-13) to depends
   * Use lsb common functions to simplify status case in havp.init
   * Added remote_fs to Required-start and Required stop in havp.init
   * Add run level 1 to default stop in havp.init
   * Bump stanards version to 3.9.5
   * Add SSLTIMEOUT to etc/havp/havp.config.in (Closes: #628613)
   * Set default clamd socket value to the correct location for Debian
    (Closes: #628615)
Checksums-Sha1:
 6fc52064c48f43bf258d3ec5a425d3d0372ca967 2076 havp_0.92a-3.dsc
 4cb06fb0c2d6b9cbee1a2e8f0ac4eea529d5415d 23472 havp_0.92a-3.debian.tar.xz
 e7808e87acae26d9d80ef6d197d6f570aed83e1b 137310 havp_0.92a-3_i386.deb
Checksums-Sha256:
 1838fc17a285ea577198273230984da626c18326831c522c525c51876b50fc7d 2076 
havp_0.92a-3.dsc
 338cb927ae0743fd1963f75b1db37c43ecee09ef4f44c65f1445b8a1dff18874 23472 
havp_0.92a-3.debian.tar.xz
 15457578bcbb825968ab75f20f572bd1047447d36453582d1094b96798d0decf 137310 
havp_0.92a-3_i386.deb
Files:
 50f5875b306e721b6c41c0f32ddd84e6 137310 net optional havp_0.92a-3_i386.deb
 3ab0337a5d1f63d2cc649a6c09744f8b 2076 net optional havp_0.92a-3.dsc
 54b7404d684031215f816f2ee6f51e0b 23472 net optional havp_0.92a-3.debian.tar.xz

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBCAAGBQJT5wnHAAoJEHjX3vua1ZrxWKUQALpXd9UhGlyInVxAlZWrr+HP
RIpXw3nSdZpxkCLcIsZ2EprwOG/PSVHHXVu2dgS6lFK7A8GLNMuZVSSsakGI/R7d
m6COqD4UZDhYaqKd+uMWAKhRUB+PBk7Bb2Jn/zHkpfR5sVssK5Uil5VNP4JFDsg3
K/bo8wQY8nhR0RfiPUFMGNYLAzGrUf6AYhsgQR4LeiUGvpkLnWoYKZtl3BVO+QlV
Ycn/HrIgZ1xcQJsXzh3Tu6XbMZkexXiyBx4/z8S8O3zzYLNs9nQ/JSRrcrA5ZkkC
38gl0zTuIRcawJ+/dResYzm8eSk75L/DhY2UczF/UqsIZPJQFIQSCSvX+lwpgZp0
0cgYZC5Nd/qO9eEVwuPsdp+jpIPeotMxdyDMQHEzZWmxSwQKaFNQ9jKM/9oGxmgm
U11McSDVnlEmDNNBavn4V0YircNm69C2mlx0oDdA+k5Y4ihTa+oJo0SYkv5kJs7K
judN98+S6QD02PByDnuHVA6PgOWrG6gXXQIo1R3MMb1wfN8zb3Va1SJsjYuiUsS1
8uO5iggiovmS2pBktL2ZV5sOm+ZTS0uPpGjyuQnNvrtr99lShwBYPpJzG4cuqBbu
K4VSPv1FIF2T1MfCbVUGtSrKuHR7Pxi+GsaWf+550Y0uIsjlzPsSdVEHpwl1nQUy
86jo7FUQ7IAsxrSUw2sG
=LUnA
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to