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(developer): Windows package installer compiler 🎺 #9607

Merged

Conversation

mcdurdin
Copy link
Member

@mcdurdin mcdurdin commented Sep 21, 2023

Fixes #9538.

Replaces kmcomp's package installer build infrastructure. While this is mostly legacy, package installers are still used in adhoc distribution scenarios, so we need to continue to support with the new kmc.

The package installer is a Windows self-extracting zip archive. The user is expected to find setup-redist.exe and keymandesktop.msi themselves, from the Keyman Developer release files. They are not included with Keyman Developer, but are available as standalone downloads from https://downloads.keyman.com/windows/.

kmc build has been updated to include two subcommands:

  • kmc build ldml-test-data
  • kmc build windows-package-installer

Both of these subcommands are a little long, but the use cases for them are fairly narrow, so I believe this is okay. kmc build-test-data has been removed, as kmc build ldml-test-data replaces it.

@keymanapp-test-bot skip

Replaces kmcomp's package installer build infrastructure. While this is
mostly legacy, package installers are still used in adhoc distribution
scenarios, so we need to continue to support with the new kmc.

The package installer is a Windows self-extracting zip archive. The user
is expected to find setup-redist.exe and keymandesktop.msi themselves,
from the Keyman Developer release files. They are not included with
Keyman Developer, but are available as standalone downloads from
https://downloads.keyman.com/windows/.

`kmc build` has been updated to include two subcommands:
* `kmc build ldml-test-data`
* `kmc build windows-package-installer`

Both of these subcommands are a little long, but the use cases for them
are fairly narrow, so I believe this is okay. `kmc build-test-data` has
been removed, as `kmc build ldml-test-data` replaces it.
@keymanapp-test-bot keymanapp-test-bot bot added the user-test-missing User tests have not yet been defined for the PR label Sep 21, 2023
@keymanapp-test-bot keymanapp-test-bot bot changed the title feat(developer): Windows package installer compiler feat(developer): Windows package installer compiler 🎺 Sep 21, 2023
@keymanapp-test-bot keymanapp-test-bot bot added this to the A17S22 milestone Sep 21, 2023
@mcdurdin mcdurdin linked an issue Sep 22, 2023 that may be closed by this pull request
@github-actions github-actions bot added the core/ Keyman Core label Sep 22, 2023
@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-missing User tests have not yet been defined for the PR label Sep 29, 2023
@mcdurdin mcdurdin marked this pull request as ready for review September 29, 2023 07:37
@mcdurdin mcdurdin modified the milestones: A17S22, A17S23 Sep 30, 2023
Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

Base automatically changed from feat/developer/remove-kmcmpdll to epic/package-metadata October 3, 2023 13:35
@mcdurdin mcdurdin merged commit f864f57 into epic/package-metadata Oct 3, 2023
2 checks passed
@mcdurdin mcdurdin deleted the feat/developer/kmc-package-windows-installer branch October 3, 2023 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat(developer): support building package installers with kmc 🎺
2 participants