[Pkg-zsh-devel] Bug#1023317: Bug#1023318: / Bug#1023317: {screen, zsh}: re-adds /usr/bin/{screen, zsh} to /etc/shells on upgrade

[Axel Beckert]

Hi Helmut,

Helmut Grohne wrote:
> On Thu, Nov 03, 2022 at 12:26:18AM +0100, Axel Beckert wrote:
> > Some more hints about how this all is meant to work and why it
> > actually works would have been appreciated, though:
> 
> Thank you for the quick reply and thank you for bringing up the lack of
> documentation.

You're welcome. And thanks for not being annoyed of my reply. :-)

> > Helmut Grohne wrote:
> > It took me quite a while to where this
> > /usr/share/debianutils/shells.d/ comes from and where it is this
> > documented,
> 
> I'm sorry for using your time in such a bad way. Consider asking for
> feedback earlier next time.

Well, you know, I first try to figure out things myself before asking
others too quickly. And I usually dig further if I have the feeling,
I'm very close to the answer. In the end you got my way through the
problem as I tend to protocol what I've done.

> > /usr/share/doc/debianutils/README.shells.gz clearly states that I
> > should use add-shell and remove-shell:
> 
> In all my work, I happen to not have run into this, but I really should.
> Thank you for pointing it out. Failing to update it is an obvious
> omission in retrospect and this is the place that should have told you
> all that you wanted to know.

Indeed.

> I have thus filed #1023380 to fix that.

Thanks a lot! Looks good to me and clearly improves the situation a lot.

> Please let me know if you miss anything else.

Maybe the man pages add-shell(8) and remove-shell(8) should mention
update-shells(8) under "SEE ALSO" or mention that there's also a
trigger-based invocation of them (with pointers then). That would have
helped me as well. (Cc'ing #1023380 for these suggestions.)

P.S.: Will work on the two issues you've reported soon-ish. Thanks
again for making us aware of them.

		Regards, Axel
-- 
 ,''`.  |  Axel Beckert <abe at debian.org>, https://people.debian.org/~abe/
: :' :  |  Debian Developer, ftp.ch.debian.org Admin
`. `'   |  4096R: 2517 B724 C5F6 CA99 5329  6E61 2FF9 CD59 6126 16B5
  `-    |  1024D: F067 EA27 26B9 C3FC 1486  202E C09E 1D89 9593 0EDE