On 2021-07-04 10:21, Colin Watson wrote: > On Sun, Jul 04, 2021 at 08:17:59AM +0000, Martin wrote: >> No harm done by `update_ssh_group_name()`, but maybe I would use >> `if ! getent group _ssh >/dev/null; then` instead of >> `if getent group ssh >/dev/null; then` just for the aesthetics (and to >> prevent postinst failure, in case both `ssh` and `_ssh` already exist). > > The version check should deal with most of this in practice, but I've > added a check for _ssh in addition (rather than "instead").
Even better, thank you!