fix: platform tag detection for Android targets#3023
Merged
Conversation
Follow up of a23a6b5 (PyO3#2992) It is recommended to go through the commit message linked above before trying to decode this. Since earlier Android was detected as Os::Linux, this extra condition was required, this should no longer be the case. So clean it up A workaround is to set _PYTHON_HOST_PLATFORM=android_{ANDROID_API_LEVEL}_{ANDROID_ARCH} for build scripts that expect that wheels are named in a certain pattern Issue originally reported in PyO3#2992 (comment)
messense
approved these changes
Feb 20, 2026
|
Thanks, I can confirm that when this commit is in a release of Maturin, |
netbsd-srcmastr
pushed a commit
to NetBSD/pkgsrc
that referenced
this pull request
Feb 22, 2026
## 1.12.4 * Upgrade memmap2 version ([#3021](PyO3/maturin#3021)) * Fix: platform tag detection for Android targets ([#3023](PyO3/maturin#3023)) * Fix: only ignore maturin-generated native libraries on all platforms ([#3025](PyO3/maturin#3025)) * Fix: ignore develop artifacts for all binding types during build ([#3026](PyO3/maturin#3026)) * Feat: support conditional cargo features based on Python version ([#3027](PyO3/maturin#3027)) ## 1.12.3 * Ci: set crt-static for riscv64 and loongarch64 musl targets ([#3009](PyO3/maturin#3009)) * Fix: support `maturin develop` on Windows ARM with x86 Python ([#3011](PyO3/maturin#3011)) * Fix: exclude `external_packages` bindings from uniffi wheels ([#3013](PyO3/maturin#3013)) * Update cargo-zigbuild to 0.22.1 ([#3015](PyO3/maturin#3015)) * Feat: build wheels from sdist with `--sdist` flag ([#3014](PyO3/maturin#3014)) * Feat: add `include-import-lib` option to bundle Windows import libraries in wheels ([#3017](PyO3/maturin#3017)) * Fix: auditwheel external lib check respects musllinux and reports symbol versions ([#3019](PyO3/maturin#3019))
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow up of a23a6b5 (#2992)
It is recommended to go through the commit message linked above before trying to decode this. Since earlier Android was detected as Os::Linux, this extra condition was required, this should no longer be the case. So clean it up
A workaround is to set
_PYTHON_HOST_PLATFORM=android_{ANDROID_API_LEVEL}_{ANDROID_ARCH}for build scripts that expect that wheels are named in a certain patternIssue originally reported in #2992 (comment)
CC @robertkirkman
Additionally, I'd appreciate if we can get a patch release with this commit for maturin, as this issue is causing our CI to fail for Python packages for Termux