Skip to content

Added support for the android native platforms - #293

Merged
garyp merged 1 commit into
streem:masterfrom
nadavgu:nadavgu/android-native
Oct 9, 2025
Merged

garyp merged 1 commit into
streem:masterfrom
nadavgu:nadavgu/android-native

Conversation

@nadavgu

@nadavgu nadavgu commented Sep 26, 2025

Copy link
Copy Markdown
Contributor

Added support for the following platforms:

  • androidNativeArm64
  • androidNativeArm32

Taken inspiration from the PR that added support for mingwX64

Ran the conformance tests for both platforms locally on an android device.
But didn't upload the modifications required to do so (modifications in the test script to also run on a connected android device using adb when a flag is passed & changes to conformance-native to support 32bit).
Tell me if I should :)

@nadavgu
nadavgu force-pushed the nadavgu/android-native branch 2 times, most recently from b66c437 to 48e77f5 Compare September 26, 2025 15:47
@nadavgu
nadavgu force-pushed the nadavgu/android-native branch from 48e77f5 to de19cfc Compare September 26, 2025 15:48
@garyp

garyp commented Oct 9, 2025

Copy link
Copy Markdown
Collaborator

Thanks @nadavgu!

Ran the conformance tests for both platforms locally on an android device.
But didn't upload the modifications required to do so (modifications in the test script to also run on a connected android device using adb when a flag is passed & changes to conformance-native to support 32bit).
Tell me if I should :)

I'll merge this PR as is, but it would be nice to have support for running the conformance tests on Android devices. We've had a long-standing issue open relating to this: #128. So even if you have an in-progress branch with your changes, it'd be helpful to have as a starting point.

@garyp
garyp merged commit 98493da into streem:master Oct 9, 2025
21 of 22 checks passed
@nadavgu

nadavgu commented Oct 10, 2025

Copy link
Copy Markdown
Contributor Author

Opened a PR for it #294 :)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants