[Pkg-puppet-devel] Bug#571130: Bug#571130: puppet-common: Package purge should remove all puppet directories

[Mathias Gug]

Hi,

On Wed, Feb 24, 2010 at 06:18:37PM -0500, micah anderson wrote:
> 
> Ok, this one scares me, only because it makes me worry that we would
> remove someone's hard work that they've spent crafting recipes in
> /etc/puppet. Purge should *only* remove the package files that the
> admins have not touched, instead of rm -rf'ing the whole thing. Or at
> least defaulting to *not* do it and a debconf question that requires
> confirmation before nuking it (like in the mysql package's removal of
> /var/lib/mysql which contains your database data).
> 
> The /var/log puppet directory clearly needs to go.
> 

Based on the feedback I've attached an updated version of the patch.

I've added /etc/puppet to puppet-common.dirs so that dpkg will automatically do
the right thing (ie /etc/puppet will be removed only if empty). /var/lib/puppet
will only be deleted if it's empty as well.

-- 
Mathias Gug
Ubuntu Developer  http://www.ubuntu.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: package-purge2.patch
Type: text/x-diff
Size: 940 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-puppet-devel/attachments/20100225/7b2a54b4/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-puppet-devel/attachments/20100225/7b2a54b4/attachment.pgp>