Hi Romain,
On 2019-09-12 23:07 +0200, Romain Francoise wrote:
> On Mon, Sep 9, 2019 at 9:27 PM Sven Joachim <svenj...@gmx.de> wrote:
>> I guess the purpose is to preserve local configuration and give the
>> local admin to the ability to remove the /usr/bin/tmux entry from
>> /etc/shells.
>
> Yes, I don't remember the details but that is apparently the intent
> and it's probably inspired in part by #477750.
I see, thanks for the pointer.
>> Looking at what other shells do, bash and dash behave the same way as
>> tmux, but only really brave souls dare to remove either of those. On
>> the other hand fish, ksh, mksh, rc, screen, tcsh and zsh call add-shell
>> unconditionally.
>
> Would it help to only call remove-shell on purge?
I think that would be correct. When filing the bug, I was worried that
leaving the entry in /etc/shells might fool an unsuspecting user to
chsh(1) to a non-existent program, but chsh does not actually let you do
this (unless you are root in which case it only warns, and then
/etc/shells is ignored anyway).
Since apparently just about everyone currently gets this wrong, I think
it would be good to discuss it on debian-devel and file bugs against the
affected packages when a solution is agreed upon.
Cheers,
Sven
