Bug#705995: squid-deb-proxy: unowned files after purge (policy 6.8, 10.8): /etc/squid-deb-proxy/*.d/10-default

2014-06-24 Thread Michael Vogt
On Thu, Jun 19, 2014 at 02:44:48PM -0700, Vagrant Cascadian wrote:
[..]
 The following patch should fix this, using the same variable names in 
 postinst:
[..]
 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...

Thanks a bunch! I merged the patch into my bzr tree and it will be
part of the next upload.

Cheers,
 Michael


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#705995: squid-deb-proxy: unowned files after purge (policy 6.8, 10.8): /etc/squid-deb-proxy/*.d/10-default

2014-06-19 Thread Vagrant Cascadian
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.0 -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


signature.asc
Description: Digital signature


Bug#705995: squid-deb-proxy: unowned files after purge (policy 6.8, 10.8): /etc/squid-deb-proxy/*.d/10-default

2013-04-23 Thread Andreas Beckmann
Package: squid-deb-proxy
Version: 0.7
Severity: important
User: debian...@lists.debian.org
Usertags: piuparts

Hi,

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-defaultnot owned


cheers,

Andreas


squid-deb-proxy_0.7.log.gz
Description: GNU Zip compressed data