[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1036016: unblock: amavisd-new/1:2.13.0-3



Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock
X-Debbugs-Cc: amavisd-new@packages.debian.org, bam@debian.org
Control: affects -1 + src:amavisd-new

Please unblock package amavisd-new

[ Reason ]

In the context of #1035654 me and Andreas Beckmann discovered nine source
packages that fail to purge without the adduser package installed. Accepting
this change would reduce that number to eight and close #1035841 in testing.

[ Impact ]

That depends on the resolution of #1035654. We want to avoid the
situation where a user removes the package, then upgrades to apt that
doesn't depend on adduser, then removes adduser and then attempts to
purge this package.

[ Tests ]

See the script I posted in #1035654.

[ Risks ]

Low risk as the diff simply is:

diff -Nru amavisd-new-2.13.0/debian/amavisd-new.postrm amavisd-new-2.13.0/debian/amavisd-new.postrm
--- amavisd-new-2.13.0/debian/amavisd-new.postrm	2023-02-23 05:59:36.000000000 +0100
+++ amavisd-new-2.13.0/debian/amavisd-new.postrm	2023-05-12 00:41:50.000000000 +0200
@@ -33,7 +33,7 @@
 		dpkg-statoverride --remove $i || true
 	done
 
-    userdel amavis
+    userdel amavis || true
 
 	echo "Removing amavis files and directories..."
 	[ -d /var/lib/amavis ] && rm -fr /var/lib/amavis

[ Checklist ]
  [x] all changes are documented in the d/changelog
  [x] I reviewed all changes and I approve them
  [x] attach debdiff against the package in testing

[ Other info ]

I'd like to get some feedback on my proposal in #1035654 and I'd like to
know whether you'd think whether I should NMU the remaining eight
packages that fail to purge without adduser with the same patch as this
package?

unblock amavisd-new/1:2.13.0-3


Reply to: