Conversation
Cube707
left a comment
There was a problem hiding this comment.
Please modify the README to mention the platform (as an untested platform)
|
I would like to have an initial test result document before merging this. Even if the platform will be listet as "no support". Please modify the POSIX Testcode to be run on your platform (but don't commit that change!), run the tests on your system and post the results. |
|
hey there, apologies for the wait - is this text report sufficient? running the manual test script also seems to work as expected |
|
+1 |
|
|
||
| - macOS | ||
| - FreeBSD / OpenBSD | ||
| - Termux |
There was a problem hiding this comment.
| - Termux | |
| - Termux (Android) |
There was a problem hiding this comment.
Maybe we should just mention Android, since this change comes from upstream and not from Termux patches. Any android build of python based on upstream v3.13 onwards includes this change.
For example, the Coding Python app on the Play Store also reports sys.platform == "android" even though it does not include Termux patches.
There was a problem hiding this comment.
Yes, If this is not specific to Termux, listing android directly should be better
Python 3.13 changed the value returned by
sys.platform()to now returnandroid, instead oflinux, on Android (such as via Termux)