[Pkg-clamav-devel] Another upload?

Scott Kitterman debian at kitterman.com
Fri Mar 27 19:36:25 UTC 2009


On Fri, 27 Mar 2009 19:30:16 +0000 Stephen Gran <sgran at debian.org> wrote:
>This one time, at band camp, Scott Kitterman said:
>> > I have a couple of inputs I'll try to get you in the morning.
>
>Something (presumably the apparmor checks in preinst) are causing:
>E: clamav-daemon: maintainer-script-uses-dpkg-status-directly preinst
>E: clamav-freshclam: maintainer-script-uses-dpkg-status-directly preinst
>
>Looking at the scripts themselves, I'm unhappy with them on
>several counts:
>
>    mkdir -p `dirname $APP_COMPLAIN` 2>/dev/null || true
>
>Calling mkdir for things under /etc feels dirty - if we want the
>directory, shouldn't we ship it in the package?
>
>    pkg_md5sum="`sed -n -e \"/^Conffiles:/,/^[^ 
]/{\\\\'$APP_CONFFILE'{s/.* //;p}}\" /var/lib/dpkg/status`"
>
>There's a dpkg-status command or something to get this sort of
>information nowadays, isn't there?  This is specifically what the
>lintian test is complaining about, so it seem worth it to do it the
>right way.
>
>  if [ -e "$APP_CONFFILE" ]; then
>    ..
>  else
>    ln -sf $APP_CONFFILE $APP_COMPLAIN
>  fi
>
>Isn't that a dangling symlink?  What's the point of that?
>

I neither did, nor did I closely examine the apparmor stuff.  I'll ask.  I 
think the dir in /etc is generally used by packages that provide apparmor 
profiles.

Don't let this hold up the next upload.  We can always merge it later.

Scott K



More information about the Pkg-clamav-devel mailing list