On Thu, Apr 04, 2013 at 04:13:51PM +0200, Gonéri Le Bouder wrote: > On Wed, Apr 03, 2013 at 09:58:43PM +0100, Jonathan Wiltshire wrote: > > On Thu, Mar 28, 2013 at 12:22:38PM +0000, Jonathan Wiltshire wrote: > > > On Tue, Mar 26, 2013 at 04:17:00PM +0100, Gonéri Le Bouder wrote: > > > > Hi, > > > > > > > > > -7 looks fine in principle. Please upload it with urgency=medium so > > > > > it can > > > > > get a final review. > > > > The package has been uploded. > > > > > > I'm sorry not to have spotted it earlier, but this change in prerm: > > > > > > + # remove the configuration file itself > > > + rm -f /etc/fusioninventory/agent.cfg > > > + rm -r /etc/fusioninventory > > > > > > Apart from the redundancy, what happens if the directory isn't there? > > > Won't > > > the purge abort? > > > > ping? > Sorry, > > Indeed, this is enough to break the postrm script. I prepare a new package.
I uploaded 2.2.3-8, this is the debdiff against 2.2.3-7:
diff -u fusioninventory-agent-2.2.3/debian/changelog
fusioninventory-agent-2.2.3/debian/changelog
--- fusioninventory-agent-2.2.3/debian/changelog
+++ fusioninventory-agent-2.2.3/debian/changelog
@@ -1,3 +1,10 @@
+fusioninventory-agent (2.2.3-8) unstable; urgency=medium
+
+ * purge: avoid postrm script failure if /etc/fusioninventory is missing
+ * Urgency medium, because it fix bug #684855 in testing
+
+ -- Gonéri Le Bouder <gon...@rulezlan.org> Thu, 04 Apr 2013 16:18:10 +0200
+
fusioninventory-agent (2.2.3-7) unstable; urgency=medium
* Drop upstream-fix-backport.diff in order to reduce the size of the
diff -u fusioninventory-agent-2.2.3/debian/fusioninventory-agent.postrm
fusioninventory-agent-2.2.3/debian/fusioninventory-agent.postrm
--- fusioninventory-agent-2.2.3/debian/fusioninventory-agent.postrm
+++ fusioninventory-agent-2.2.3/debian/fusioninventory-agent.postrm
@@ -11,7 +11,7 @@
# remove the configuration file itself
rm -f /etc/fusioninventory/agent.cfg
- rm -r /etc/fusioninventory
+ [ -d /etc/fusioninventory ] && rmdir --ignore-fail-on-non-empty
/etc/fusioninventory
# and finally clear it out from the ucf database
if which ucf >/dev/null; then
pgpXMRXJAblkr.pgp
Description: PGP signature
