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

Change active VTE restriction implementation to a xmodmap script #178

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

yfyf
Copy link
Collaborator

@yfyf yfyf commented Sep 24, 2024

⚠️🐃 yak-shaving alert! 🐃 ⚠️

It was very annoying to wait for xorg to be rebuilt just to make this small restriction, so I had to shave this yak. Perhaps not the most elegant approach, but it works and avoid needless re-compilation. Easier to maintain and more "pluggable" as well.

Manually tested via run-in-vm and QEMU's sendkey. Ctrl+Shift+F12 continues to work. As before, Ctrl+Alt+F8 + Ctrl-Alt+F{N} is a hack to "escape" the restriction.

This could actually be limited at the VTE/kernel level as well in a similar way using something like

    dumpkeys | grep .. Console_{N} .. | sed .. | loadkeys

but not sure we want that.

Checklist

  • Changelog updated
  • Code documented
  • User manual updated

…script

It was very annoying to wait for xorg to be rebuilt just to make this
small restriction, so I had to shave this yak. Perhaps not the most
elegant approach, but it works and avoid needless re-compilation.
Easier to maintain and more "pluggable" as well.

Manually tested via `run-in-vm` and QEMU's sendkey. Ctrl+Shift+F12
continues to work. As before, Ctrl+Alt+F8 + Ctrl-Alt+F{N} is a hack to
"escape" the restriction.

This could actually be limited at the VTE/kernel level as well in
a similar way using something like

        dumpkeys | grep .. Console_{N} .. | sed .. | loadkeys

but not sure we want that.
@yfyf yfyf added the reviewable Ready for initial or iterative review label Sep 24, 2024
Copy link
Member

@knuton knuton left a comment

Choose a reason for hiding this comment

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

Nice 👌

Verified that VT1 can not be accessed directly, VT8 can, VT7 can. (Also, VT1 can from VT8, as before.)

@knuton knuton merged commit a9e2269 into dividat:main Sep 24, 2024
5 checks passed
@knuton knuton removed the reviewable Ready for initial or iterative review label Sep 24, 2024
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.

2 participants