keyman icon indicating copy to clipboard operation
keyman copied to clipboard

refactor(android): Use builder scripts

Open darcywong00 opened this issue 3 years ago • 1 comments

This updates the top android/build.sh to use the new builder pattern.

Targets: :engine Keyman Engine for Android :app - Keyman for Android :samples - Sample apps (KMSample1, KMSample2, KeyboardHarness) :fv - OEM FirstVoices app

The default dev build would change from: from ./build.sh -debug to ./build.sh build --debug

TODO:

  • [x] Update CI Android Test-14.0 and Test-14.0: Samples and Test projects configs
  • [ ] Update CI Android Release-14.0 configs (possibly have a single config step for ./build.sh clean build:engine build:app build:samples build:fv --CI --download-resources and then skip other legacy steps if exist android/.build-builder. Followed by ./build.sh publish --CI
  • [ ] Test/verify publish works

@keymanapp-test-bot skip

darcywong00 avatar Oct 05 '22 03:10 darcywong00

User Test Results

Test specification and instructions

User tests are not required

keymanapp-test-bot[bot] avatar Oct 05 '22 03:10 keymanapp-test-bot[bot]

See #7474 for Web's rework & the accompanying Android build-script tweaks.

Well, at least in their prospective form. There's still a bit more work to do there, too.

jahorton avatar Oct 18 '22 08:10 jahorton

~I plan to abandon this PR and resume Android builder script activity off of @jahorton 's KeymanWeb builder script work (maybe #7474)~

Android TODO's

  • [x] Incorporate dependency syntax for Android projects:
Android Project Targets Dependency
KMEA app (itself) @../../web (or call existing KeymanWeb build script if builder work not ready)
KMAPro app @../kmea
Samples app @../../kmea
KeyboardHarness app @../../kmea
  • [x] OEM/FirstVoices app will be split out of the Android build.sh script and use its own builder script with dependecy on KMEA

darcywong00 avatar Oct 25 '22 04:10 darcywong00

Eh, you've kept it in draft. A rebase should be fine for when the time comes; I don't expect extreme merge conflicts to arise.

jahorton avatar Oct 25 '22 04:10 jahorton

Changes in this pull request will be available for download in Keyman version 17.0.66-alpha

keyman-server avatar Mar 13 '23 18:03 keyman-server