Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop shipping /etc/zsh/zlogout to avoid clearing terminal screen #158

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mika
Copy link
Member

@mika mika commented Oct 2, 2024

/etc/zsh/zlogout causes the clearing of the terminal screen. It was introduced in commit c31db6c back in 2006, though several of us Grml developers actually don't like this behavior.

While working on this, had to touch dpkg-divert, again, and had to dig deeper, quoting from the resulting commit:

debian/preinst: drop deprecated dpkg-divert
    
Usage of backup_conffile was introduced for moving diverted configuration
files from package grml-etc to grml-etc-core. We seem to have migrated
this in 2006, so there really shouldn't any users out there anymore. :)
  
Also the file /etc/skel/.zshrc is no longer shipped with Debian's zsh
package since v4.3.0-dev-2-1, which dates back to 2011-02-11 (see git
rev 78ab02f in https://salsa.debian.org/debian/zsh.git). Even oldoldstable
has v5.7.1, so finally drop this workaround.

So take this as chance to clean up debian/preinst at least a bit.

So take this as chance to clean up debian/preinst at least a bit.

/etc/zsh/zlogout causes the clearing of the terminal screen.
It was introduced in commit c31db6c back in 2006, though
several of us Grml developers actually don't like this behavior.
Usage of backup_conffile was introduced for moving diverted configuration
files from package grml-etc to grml-etc-core. We seem to have migrated
this in 2006, so there really shouldn't any users out there anymore. :)

Also the file /etc/skel/.zshrc is no longer shipped with Debian's zsh
package since v4.3.0-dev-2-1, which dates back to 2011-02-11 (see git
rev 78ab02f in https://salsa.debian.org/debian/zsh.git). Even oldoldstable
has v5.7.1, so finally drop this workaround.

So take this as chance to clean up debian/preinst at least a bit.
@mika mika requested review from zeha and jkirk October 2, 2024 21:51
@zeha
Copy link
Member

zeha commented Oct 3, 2024

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants