[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