On Fri, 27 Mar 2009 19:30:16 +0000 Stephen Gran <[email protected]> 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 _______________________________________________ Pkg-clamav-devel mailing list [email protected] http://lists.alioth.debian.org/mailman/listinfo/pkg-clamav-devel
