[Pkg-clamav-devel] Another upload?

[Scott Kitterman]

> I have a couple of inputs I'll try to get you in the morning.
>
> Scott K

One user (hi leonel) of the test package I made for Ubuntu of 0.95-1
reported:


ScottK:  Setting up clamav-base (0.95+dfsg-1~ppa2) ...
chown: cannot access `/var/run/clamav': No such file or directory
dpkg: error processing clamav-base (--configure):
 subprocess post-installation script returned error exit status 1
leonel: Does /var/run/clamav exist?
If it does, then I'm thinking apparmor.
ScottK: was an upgrade and the dir didn't existed
ScottK: created and upgrade went fine

On Ubuntu /var/run defaults to a tempfs (and this is a config the package
supports).  I looked on the postinst and while the postinst chown's
/var/run/clamav, I don't see where it ensures it's been created by that
time.  So I think this needs to be addressed.  I confess not to have had a
lot of time to look into this, so I might have missed something.

Several months ago I added make check to debian/rules in Ubuntu.  Except
for a lot of failures from the valgrind checks (which led to upstream
disabling the valgrind checks by default), I haven't had any problems.  I
think running the tests during the build is a generally good practice and
now, at the start of a release cycle, is a good time for Debian to do this
too.

Finally, we've discussed before adding apparmor as a suggests (it doesn't
exist in the Debian archive, but Debian users may have added it locally)
and incorporating the Ubuntu apparmor profile in the package.  The bind9
package in Debian also does this, so there is precedent.  Patch attached.

That's all I have.  This covers all of the current Debian/Ubuntu diff in
the package.

Scott K
-------------- next part --------------
A non-text attachment was scrubbed...
Name: apparmor.diff
Type: text/x-diff
Size: 8890 bytes
Desc: not available
Url : http://lists.alioth.debian.org/pipermail/pkg-clamav-devel/attachments/20090327/f2d58a7e/attachment-0001.diff