Skip to content

feat(mac): add support for legacy kmp.inf packages 🍎 - #16551

Open
sgschantz wants to merge 8 commits into
epic/mac-configfrom
feat/mac/kmp-inf
Open

feat(mac): add support for legacy kmp.inf packages 🍎#16551
sgschantz wants to merge 8 commits into
epic/mac-configfrom
feat/mac/kmp-inf

Conversation

@sgschantz

@sgschantz sgschantz commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

If a kmp.json file cannot be found within a package, read the kmp.inf file and use it to load the file rather than throwing an error.

Fixes: #16485

Build-bot: release:mac
Test-bot: skip

basing on Objective C code

added default for readMe and Help files
added validation for required fields
@sgschantz sgschantz added this to the B19S1 milestone Sep 10, 2026
@sgschantz sgschantz self-assigned this Sep 10, 2026
@sgschantz sgschantz added the mac/ label Sep 10, 2026
@sgschantz sgschantz added this to Keyman Sep 10, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Keyman Sep 10, 2026
@keymanapp-test-bot

keymanapp-test-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

@keymanapp-test-bot keymanapp-test-bot Bot changed the title feat(mac): add support for legacy kmp.inf packages feat(mac): add support for legacy kmp.inf packages 🍎 Sep 10, 2026
@github-actions github-actions Bot added the feat label Sep 10, 2026
@sgschantz sgschantz linked an issue Sep 10, 2026 that may be closed by this pull request
8 tasks
Base automatically changed from feat/mac/localize to epic/mac-config September 10, 2026 17:53
sgschantz and others added 4 commits September 11, 2026 13:31
@sgschantz

Copy link
Copy Markdown
Contributor Author

@mcdurdin, do you know any good packages to test this with? The only ones I have found could not be loaded for other reasons apart from reading the kmp.inf. I'd like to test with something official instead of just keyboard packages where I removed the kmp.inf.

@sgschantz
sgschantz marked this pull request as ready for review September 11, 2026 18:19
@sgschantz
sgschantz requested a review from mcdurdin September 11, 2026 20:25
@keyman-server keyman-server modified the milestones: B19S1, B19S2 Sep 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

bug(mac): add kmp.inf support

2 participants