Skip to content

fix(android): Update sentry-android dependency to 7.22.6 for 16 KB page alignment 🍒 🏠#14560

Merged
darcywong00 merged 1 commit intostable-18.0from
fix/android/sentry-7
Aug 21, 2025
Merged

fix(android): Update sentry-android dependency to 7.22.6 for 16 KB page alignment 🍒 🏠#14560
darcywong00 merged 1 commit intostable-18.0from
fix/android/sentry-7

Conversation

@darcywong00
Copy link
Copy Markdown
Contributor

Addresses #14423 for stable-18.0 (similar to #14559)

https://developer.android.com/guide/practices/page-sizes

This updates the Sentry dependency to 7.22.6 for 16 KB page alignment. stable-18.0 couldn't get the most recent 8.19.1 due to older Gradle dependencies, but the 16 KB alignment is resolved.

Using the APK analyzer tool in Android Studio, I was able to verify the Keyman apk and internal .so libraries no longer have the alignment warnings.

image

Note: Because KMEA is an .aar project, the android/KMEA/build.gradle ended up not needing the Sentry dependency because it's handled in android/KMEA/app/build.gradle. Only app-level projects need the sentry-android-gradle-plugin.

User Testing

Setup - Create an Android emulator that uses a Pre-Release 16 KB Page Size image

  • From Android Studio - Device Manager --> Add New Device --> Create Virtual Device

  • Select Pixel 8a phone form factor -->

    • API 36.0 Baklava
    • Services: Google Play Store
    • download the "Pre-Release 16 KB Page Size Google Play Intel_x86_64 Atom System image API 36.0" and select it
  • On this emulator, install the PR build of Keyman for Android

  • TEST_KEYBOARD_INSTALL - Verifies Keyman functions and can install keyboard

  1. Launch Keyman
  2. From the "Get Started" menu, install sil_cameroon_qwerty keyboard from keyman.com.
  3. Verify you can select a language during the keyboard installation and the sil_cameroon_qwerty keyboard can install
  4. Return to Keyman and verify the sil_cameroon_qwerty keyboard functions

@darcywong00 darcywong00 added this to the A19S10 milestone Aug 20, 2025
@github-project-automation github-project-automation bot moved this to Todo in Keyman Aug 20, 2025
@keymanapp-test-bot keymanapp-test-bot bot added has-user-test user-test-required User tests have not been completed labels Aug 20, 2025
@keymanapp-test-bot
Copy link
Copy Markdown

keymanapp-test-bot bot commented Aug 20, 2025

@keymanapp-test-bot keymanapp-test-bot bot changed the title fix(android): Update sentry-android dependency to 7.22.6 for 16 KB page alignment 🍒 fix(android): Update sentry-android dependency to 7.22.6 for 16 KB page alignment 🍒 🏠 Aug 20, 2025
@dinakaranr dinakaranr self-assigned this Aug 20, 2025
@dinakaranr
Copy link
Copy Markdown

Test Results

Prerequisites setup:
Created an Android virtual device.
Pixel 8a Phone
release: Baklava
API level: Baklava
ABI: x86_64
Target: Android API baklava(16 KB page size)

I tested this PR in the "Keyman-18.0.239-test-14560" build on Android baklava preview, API 36(Emulator). Here I am sharing my observation.

  • TEST_KEYBOARD_INSTALL (Passed):
  1. Install the Keyman-18.0.239-test-14560.apk build in the Android device or emulator.
  2. Accept all the Android permission requests for storage.
  3. Open the Keyman app.
  4. On the "Get Started" dialog.
  5. Install the "sil_cameroon_qwerty" keyboard by clicking the "Add a keyboard for your language" --> install from keyman.com.
  6. Verified that the "sil_cameroon_qwerty" keyboard installed with selected language during the keyboard installation.
  7. Return to Keyman app.
  8. Verified that the sil_cameroon_qwerty keyboard functions works well.

@keymanapp-test-bot keymanapp-test-bot bot removed the user-test-required User tests have not been completed label Aug 20, 2025
@dinakaranr dinakaranr removed their assignment Aug 20, 2025
Copy link
Copy Markdown
Contributor

@ermshiperete ermshiperete left a comment

Choose a reason for hiding this comment

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

LGTM

@darcywong00 darcywong00 merged commit 5aa9975 into stable-18.0 Aug 21, 2025
7 checks passed
@darcywong00 darcywong00 deleted the fix/android/sentry-7 branch August 21, 2025 00:45
@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Aug 21, 2025
@keyman-server
Copy link
Copy Markdown
Collaborator

Changes in this pull request will be available for download in Keyman version 18.0.239

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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants