feat: add automatic Android API Level detection via getprop - #3153
Open
MartheusX wants to merge 1 commit into
Open
feat: add automatic Android API Level detection via getprop#3153MartheusX wants to merge 1 commit into
MartheusX wants to merge 1 commit into
Conversation
Member
|
I don't believe this is strictly required, Though it does not hurt to include this, but please verify the claim. |
Author
|
"I have verified the claim on my another device (Xiaomi Poco F3, Android 13, API 33). |
Member
|
So you are saying the lower API level 24 does not work on Termux, any errors? Could you also check the output of |
|
On Fairphone 4 LineageOS 23.2: On 08/06/2026 with Python 3.13: getprop ro.build.version.sdkpip install -U pydantic-coreOutput:Source: Benjamin_Loison/pydantic-core/issues/2 export ANDROID_API_LEVEL=$(getprop ro.build.version.sdk)seems to have solved the issue according to my notes. python3 -m pip debug --verboseOutput: |
Member
|
@Benjamin-Loison |
|
I had struggle establishing my Fairphone 4 environment for tens of minutes, so if you are fine with it, the following continues the investigation with Virtual Machine Manager LineageOS 23.0 virtual machine Termux: Personal notes:getpropEnvironment setup:pip install pydantic-coreOutput:apt install -y rustcOutput:apt install -y rustupOutput:apt install -y rustOutput:time pip install --verbose pydantic-coreThis output may not be the right one. time pip install pydantic-coreOutput:So I reproduce the mentioned error. export ANDROID_API_LEVEL=16
pip install --verbose pydantic-coreOutput:getprop ro.build.version.sdkclang -dumpmachineOutput: |
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.
Description
Currently,
maturinrequires theANDROID_API_LEVELenvironment variable to be explicitly set when building on-device in environments like Termux. If missing, the build fails even though the host system is a live Android instance where this information is readily available.This PR introduces a fallback mechanism:
ANDROID_API_LEVELand other clues (linker/CC) are missing, it attempts to querygetprop ro.build.version.sdk.Motivation
I encountered this issue while trying to install tools in Termux on my Redmi Note 8 Pro. Automating this step removes a manual barrier for many mobile developers.
Testing