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

feat(windows): Add -boot switch to kmshell #12994

Open
rc-swag opened this issue Jan 23, 2025 · 0 comments · May be fixed by #12995
Open

feat(windows): Add -boot switch to kmshell #12994

rc-swag opened this issue Jan 23, 2025 · 0 comments · May be fixed by #12995
Assignees
Milestone

Comments

@rc-swag
Copy link
Contributor

rc-swag commented Jan 23, 2025

See full discussion at #12956 more details after this comment.

kmshell -s means 'silent' so that means no UI is intended to be shown there... We could change that to kmshell.exe -boot, but we'd have to fixup existing installations -- we'd need to make sure that that change was applied during the upgrade (this is quite doable, just more work and testing).

I think that'd be good because it'd be clearer than the current -s which could be used any time and is ambiguous.

Originally posted by @mcdurdin in #12956 (comment)

A registry key is set Windows run location to have -s this will need to change to -boot.
initprog.pas will handle -boot. Then an audit of how -s runs and how we in most cases will want -boot to follow the same execution path. At least one exception will be prompting the user to install and update if it is available and ready to install. There may be other checks in the RunKMCOM call also.

@rc-swag rc-swag self-assigned this Jan 23, 2025
@rc-swag rc-swag moved this to Todo in Keyman Jan 23, 2025
@rc-swag rc-swag added this to the A18S20 milestone Jan 23, 2025
@rc-swag rc-swag linked a pull request Jan 23, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

1 participant