[Pkg-clamav-devel] Another upload?

Stephen Gran sgran at debian.org
Fri Mar 27 19:30:16 UTC 2009


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?

Cheers,
-- 
 -----------------------------------------------------------------
|   ,''`.                                            Stephen Gran |
|  : :' :                                        sgran at debian.org |
|  `. `'                        Debian user, admin, and developer |
|    `-                                     http://www.debian.org |
 -----------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-clamav-devel/attachments/20090327/e00018f1/attachment.pgp 


More information about the Pkg-clamav-devel mailing list