Skip to content

grml-setkeyboard: deprecate #21

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

Merged
merged 1 commit into from
Dec 13, 2024
Merged

grml-setkeyboard: deprecate #21

merged 1 commit into from
Dec 13, 2024

Conversation

zeha
Copy link
Member

@zeha zeha commented Dec 7, 2024

Point users towards grml-lang and dpkg-reconfigure keyboard-configuration instead.

Closes: #10

@zeha zeha requested review from mika and jkirk December 7, 2024 16:50
@zeha zeha self-assigned this Dec 7, 2024
@@ -40,7 +40,7 @@ case $retval in
exec grml-setlang
fi
if [ "$MENU" = "grml-setkeyboard" ]; then
exec grml-setkeyboard
exec dpkg-reconfigure keyboard-configuration
Copy link
Member

Choose a reason for hiding this comment

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

Maybe I'm holding it wrong, but tried to test this inside a VM by choosing German (Austria) keyboard, but doesn't seem to end up with such a keyboard setting then (tested inside tty1`), not claiming that my environment might be the problem, but I'd like further verification/testing yet before shipping that :)

Copy link
Member Author

@zeha zeha Dec 11, 2024

Choose a reason for hiding this comment

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

Thanks for testing :-/

Apparently console-setup doesn't bother to apply the new settings after debconf exits. I'll spare you the details 😱

Added a restart of console-setup.service, that seems to do it.

@zeha zeha force-pushed the deprecate-setkeyboard branch from 11ddec6 to f9d8e95 Compare December 11, 2024 01:35
@mika mika self-requested a review December 13, 2024 14:40
Copy link
Member

@mika mika left a comment

Choose a reason for hiding this comment

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

Wasn't aware of this console-setup.service restart behavor, TIL :)

Point users towards grml-lang and dpkg-reconfigure keyboard-configuration
instead.

Closes: #10
@zeha zeha force-pushed the deprecate-setkeyboard branch from f9d8e95 to 9ac2d55 Compare December 13, 2024 15:29
@mika mika self-requested a review December 13, 2024 15:36
Copy link
Member

@mika mika left a comment

Choose a reason for hiding this comment

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

Thx!

@zeha zeha merged commit f6baeb9 into master Dec 13, 2024
@zeha zeha deleted the deprecate-setkeyboard branch December 13, 2024 15:40
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.

grml-setkeyboard: configures non-existing /etc/sysconfig/keyboard
2 participants