Control: tags 705995 patch

On Tue, Apr 23, 2013 at 12:12:19PM +0200, Andreas Beckmann wrote:
> during a test with piuparts I noticed your package left unowned files on
> the system after purge, which is a violation of policy 6.8 (or 10.8):
> 
> http://www.debian.org/doc/debian-policy/ch-maintainerscripts.html#s-removedetails
> 
> Filing this as important as having a piuparts clean archive is a release
> goal since lenny.
> 
> >>From the attached log (scroll to the bottom...):
> 
> 0m32.3s ERROR: FAIL: Package purging left files on system:
>   /etc/squid-deb-proxy/        owned by: squid-deb-proxy
>   /etc/squid-deb-proxy/allowed-networks-src.acl.d/     owned by: 
> squid-deb-proxy
>   /etc/squid-deb-proxy/allowed-networks-src.acl.d/10-default   not owned
>   /etc/squid-deb-proxy/mirror-dstdomain.acl.d/         owned by: 
> squid-deb-proxy
>   /etc/squid-deb-proxy/mirror-dstdomain.acl.d/10-default       not owned
>   /etc/squid-deb-proxy/pkg-blacklist.d/        owned by: squid-deb-proxy
>   /etc/squid-deb-proxy/pkg-blacklist.d/10-default      not owned

The following patch should fix this, using the same variable names in postinst:

--- squid-deb-proxy.postrm~     2014-01-25 13:26:19.000000000 -0800
+++ squid-deb-proxy.postrm      2014-06-19 14:36:12.485579342 -0700
@@ -2,8 +2,13 @@
 
 set -e
 
+USER_NETWORKS=/etc/squid-deb-proxy/allowed-networks-src.acl.d/10-default
+USER_ALLOWED_DESTINATIONS=/etc/squid-deb-proxy/mirror-dstdomain.acl.d/10-default
+PKG_BLACKLIST_IN=/etc/squid-deb-proxy/pkg-blacklist.d/10-default
+
 if [ "$1" = "purge" ]; then
    rm -f /etc/squid-deb-proxy/autogenerated/*
+   rm -f "$USER_NETWORKS" "$USER_ALLOWED_DESTINATIONS" "$PKG_BLACKLIST_IN"
 fi
 
 # remove 


Though simply "rm -f /etc/squid-deb-proxy/*.d/10-default" would be simpler, at
risk of deleting some randomly created file dropped in some other .d dir and
happened to be named 10-default...


live well,
  vagrant

Attachment: signature.asc
Description: Digital signature

Reply via email to