control: -1 +pending
On Sun, Jul 21, 2019 at 01:27:38PM +0200, Laurent Bigonville wrote:
>
> The cronjob (/etc/cron.d/e2scrub_all) file is a conffile, that means
> that if the package is removed without being purge, the cronjob will
> still be installed, but the executable will not.
>
> The cronjob should test the presence of the executable.
Thanks for pointing this out. The following will be in the next
e2fsprogs release.
- Ted
commit 2e0ad4432898e13a21db3b8f76c629b19e01cadc
Author: Theodore Ts'o <ty...@mit.edu>
Date: Sun Jul 21 13:13:24 2019 -0400
e2scrub_all_cron: check to make sure e2scrub_all
Since e2scrub_all.cron is marked as a config file, it can hang around
after the package is removed, in which case e2scrub_all might not be
present. So check to make sure e2scrub_all exists before trying to
execute it.
Addresses-Debian-Bug: #932622
Signed-off-by: Theodore Ts'o <ty...@mit.edu>
Reported-by: Laurent Bigonville <bi...@debian.org>
diff --git a/scrub/e2scrub_all_cron.in b/scrub/e2scrub_all_cron.in
index f9cff878..fcfe415f 100644
--- a/scrub/e2scrub_all_cron.in
+++ b/scrub/e2scrub_all_cron.in
@@ -62,6 +62,7 @@ on_ac_power() {
return 0
}
+test -e @root_sbindir@/e2scrub_all || exit 0
test -e /run/systemd/system && exit 0
on_ac_power || exit 0
