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

xl console with replace escapes does not inform terminal apps that it is restricted #9823

Open
ben-grande opened this issue Mar 4, 2025 · 0 comments
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: other needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information.

Comments

@ben-grande
Copy link

Qubes OS release

Qubes OS 4.2

Brief summary

The patch 1009-tools-xenconsole-replace-ESC-char-on-xenconsole-outp.patch is nice, but it doesn't inform applications the terminal is dumb.

Steps to reproduce

  1. sudo xl console QUBE
  2. Try to use backspace, zsh command-line completion, zsh syntax highlighting

Expected behavior

Backspace works, colors are not applied.

Actual behavior

Backspace doesn't work, escape characters messes the screen, pain with Zsh completion and syntax highlighting.

Additional information

Now it is set to TERM=vt220, it would be better if it was set to dumb when replace escapes is used.

I have accessed to terminals that inform they are vt220 and most of the escape sequences work there, so I cannot and should not disable capabilities because of that.

tty is /dev/hvc0, although it seems super specific to a console, I believe it is best to use TERM to inform the correct capabilities of the terminal.

https://invisible-island.net/ncurses/terminfo.src.html#tic-dumb

When it is dumb, backspace works.

@ben-grande ben-grande added the P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. label Mar 4, 2025
@ben-grande ben-grande changed the title xl console path to replace escapes does not inform qube terminal apps that it is restricted xl console with replace escapes does not inform terminal apps that it is restricted Mar 4, 2025
@andrewdavidwong andrewdavidwong added C: other needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. affects-4.2 This issue affects Qubes OS 4.2. labels Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-4.2 This issue affects Qubes OS 4.2. C: other needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: default Priority: default. Default priority for new issues, to be replaced given sufficient information.
Projects
None yet
Development

No branches or pull requests

2 participants