Comment 22 for bug 1762952

Revision history for this message
Gunnar Hjalmarsson (gunnarhj) wrote :

Thanks for looking, Łukasz.

Right, it didn't build in PPA for me without those. As regards 'locales', according to the build log it looked for and failed to find /usr/share/i18n/charmaps. As regards the extra install directory in d/rules, it's obviously needed if the conditions for running the subsequent commands are satisfied. Can't explain why it differs from the archive uploads. I just run "debuild -S" and dput'ed (not from a clean chroot, though).

I think that $1 is always "configure" at the execution of postinst. Please note that the original code checks for precisely that. I struggled a bit to find a way to check whether we are upgrading, and the proposed way proved to work when I tested. But I'm not a Debian packaging wizard, and would be just as happy if there is a simpler way to check for it.

Actually, the fact that keyboard-configuration and g-c-c both write to /etc/default/keyboard, while they have different ideas of what it should contain, appears to be a bit odd. My proposal only addresses the most obvious issue as reported in this bug, i.e. it prevents the keyboard configuration from being changed without the user having asked for it.