[SCM] crtmpserver/master: Add postrm script for crtmpserver-apps.

Alessio Treglia alessio at debian.org
Tue Apr 24 00:06:43 UTC 2012


This doesn't look sane to me:

On Sat, Apr 14, 2012 at 9:45 PM,  <jet-guest at users.alioth.debian.org> wrote:
> +do_remove() {
> +       for app in $APPLICATIONS; do
> +               if [ -f "/etc/crtmpserver/applications/$app.lua" ]; then
> +                       compare "/etc/crtmpserver/applications/$app.lua" \
> +                               "/usr/share/doc/crtmpserver-apps/examples/$app.lua" \
> +                               && rm "/etc/crtmpserver/applications/$app.lua"

Why do you try to remove those files?
The crtmpserver-apps package seems to own no conffiles.

> +               fi
> +       done
> +       rmdir /etc/crtmpserver/applications

This is Severity: grave as it means removing user's files and may potentially
cause data loss!
If you really want to provide crtmpserver-apps with conffiles, you should
install them clean way (e.g. dh_install) and then manage their removal
with dpkg-maintscript-helper.

-- 
Alessio Treglia          | www.alessiotreglia.com
Debian Developer         | alessio at debian.org
Ubuntu Core Developer    | quadrispro at ubuntu.com
0416 0004 A827 6E40 BB98 90FB E8A4 8AE5 311D 765A



More information about the pkg-multimedia-maintainers mailing list