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

Add KB article on migrating from KMFL to Keyman for Linux #1335

Merged
merged 2 commits into from
Jun 27, 2024

Conversation

darcywong00
Copy link
Contributor

Addresses #94 on helping Linux users migrate from KMFL to Keyman for Linux

First commit was @ermshiperete 's answer on the community site forum (same response on Linux-user group.

Additional commits for formatting, and tweaking the section about a kmc compilation error.

@darcywong00 darcywong00 added this to the A18S5 milestone Jun 26, 2024
@ermshiperete
Copy link
Contributor

See also keymanapp/keyman#11857

@darcywong00 darcywong00 merged commit 2defdbe into master Jun 27, 2024
3 checks passed
@darcywong00 darcywong00 deleted the kb-migrate-kmfl branch June 27, 2024 09:25
@ermshiperete
Copy link
Contributor

Just had a look at the KB page and noticed that the last paragraph ("To properly do this without an error, ...") is a bit misleading when it comes under the "Use Keyman for Linux to install the .kmp keyboard package" section header. We should either move that paragraph above the section header, or add a new section header above this paragraph.

Comment on lines +68 to +69
Disregard the error: `"TypeError: Cannot read properties of undefined (reading 'description')"`, as the compiler will already have created the .kmp file in the `build`
subdirectory.
Copy link
Member

Choose a reason for hiding this comment

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

To properly do this without an error, you'll have to add some additional fields to the .kps file. It's easiest to let the `kmconvert` tool generate a template which will contain all necessary the fields:

```bash
kmconvert template -id mykeyboard
Copy link
Member

Choose a reason for hiding this comment

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants