Bug#782156: grub-xen: unowned files after purge (policy 6.8, 10.8): /boot/grub/{locale, x86_64-xen}/* and some more

[Ian Campbell]

On Wed, 2015-04-08 at 18:16 +0200, Andreas Beckmann wrote:
[...]
> This seems to be the only grub-* package leaving (lots of) stuff
> around after purge.

Looking at the postinst/postrm code it seems that actually the inverse
is true, only grub-pc is cleaning up after itself (and then only
conditionally based on debconf).

Most of the other grub-* are either !x86 (which I think piuparts doesn't
test yet) or do not actually install any grub binaries for some reason
(e.g. the postinst for grub-efi-* checks for existence
of /boot/efi/EFI/debian, which I bet doesn't exist in the piuparts
chroot).

Actually grub-efi-ia32|grub-efi-amd64 do partially clean up, but only
the /boot/grub/unicode.pf2 which is installed by the postinst outside
the /boot/efi/EFI/debian check.

Anyway, IIRC when I was doing the grub-xen packaging stuff Colin
indicated that it was considered a feature not a bug that stuff was left
in /boot on purge, essentially because removing them might render the
system unbootable.

IMHO either /boot/grub/* should be whitelisted for piuparts (based on
the above argument regarding making systems unbootable) or the existing
debconf template and postrm code for grub-pc should be applied to all
grub-* and not just extended to cover grub-xen.

BTW, the grub-pc debconf question grub-pc/postrm_purge_boot_grub
defaults to false, meaning it shouldn't be cleaning up after itself. I'm
not sure then why it apparently passes piuparts...

Ian.