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

Do not overwrite escape sequences for xterm like #140

Merged
merged 1 commit into from
Mar 12, 2024

Conversation

bitstreamout
Copy link
Member

terminals, that is \e[1~ and \e[4~ are used in vt220 and similar terminals only. Do not overwrite those escape sequences even if XTerm have a vt220 mode.

This makes the choice of the TERM variable for ssh and PuTTY users easier.

@lnussel lnussel self-requested a review March 5, 2024 13:45
Copy link
Member

@lnussel lnussel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you have a strange pull request commit in there. need to rebase

terminals, that is \e[1~ and \e[4~ are used in vt220
and similar terminals only.  Do not overwrite those
escape sequences even if XTerm have a vt220 mode.

This makes the choice of the TERM variable for ssh
and PuTTY users easier.

Signed-off-by: Werner Fink <werner@suse.de>
@bitstreamout
Copy link
Member Author

Now this tree is rebased

@lnussel lnussel merged commit d960a92 into openSUSE:master Mar 12, 2024
3 checks passed
@bitstreamout bitstreamout deleted the inputrc4vt220 branch June 14, 2024 12:12
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.

None yet

2 participants