-
Notifications
You must be signed in to change notification settings - Fork 327
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
Always use VT_OPENQRY #1896
base: develop
Are you sure you want to change the base?
Always use VT_OPENQRY #1896
Conversation
sizeof(char *) wasn't the intention there.
There is no other way to reliably grab a VT for use.
The whole concept doesn't make sense with VT_OPENQRY.
The PR works, but does unfortunately not fix the issue. It's broken at a lower level, From journalctl:
|
Apparently |
Reported as systemd/systemd#31909 |
Sorry to mention this, but there is a reason I fixed #1650: I want sddm to run on a specific VT. Removing the feature entirely renders SDDM unusable for me. |
The option to use a specific VT only ever works in the specific case of rootful X with an X desktop inside, everything else needs dynamic allocation. You can influence dynamic allocation to reserve lower VTs though and get SDDM to use a specific one that way. |
I plan to make some changes to this PR anyway (open the tty in the daemon process), which should make it easier to reintroduce SDDM_INITIAL_VT in a way that actually works though. |
There is no other way to reliably grab a VT for use.
+ bonus tty fix.
Draft because not tested yet.