Created on 2018-01-23 17:16 by vstinner, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Pull Requests |
|
|
|
| URL |
Status |
Linked |
Edit |
| PR 5287 |
closed |
vstinner,2018-01-23 21:08 |
|
| PR 5288 |
closed |
vstinner,2018-01-23 23:55 |
|
| Messages (6) |
|
|
| msg310511 - (view) |
Author: STINNER Victor (vstinner) *  |
Date: 2018-01-23 17:16 |
| Currently, sys.platform and os.name are equal to 'linux' on Android. While Android uses the Linux kernel, the operating system is not a regular Linux. The libc (bionic) is very different than the regular glibc, the filesystem is organized very differently, etc. I propose to change sys.platform and os.name to 'android' when running on Android. |
|
|
| msg310516 - (view) |
Author: STINNER Victor (vstinner) * |
Date: 2018-01-23 17:38 |
| Wait, os.name is 'posix' on Android. That's fine in fact. Only sys.platform should be updated. |
|
|
| msg310537 - (view) |
Author: STINNER Victor (vstinner) * |
Date: 2018-01-24 00:12 |
| The idea was first discussed on python-dev: https://mail.python.org/pipermail/python-dev/2018-January/151874.html me: "It seems like sys.platform == 'android' would be more appropriate since Android is not Linux: different libc, different filesystems, etc." https://mail.python.org/pipermail/python-dev/2018-January/151887.html Brett Cannon: "I've had a similar thought myself.". |
|
|
| msg310543 - (view) |
Author: STINNER Victor (vstinner) * |
Date: 2018-01-24 00:42 |
| A similar change was proposed by pmp-p on MicroPython: https://github.com/micropython/micropython/pull/3564 His use case is to use ffi module on Android (see the PR for more information), currently ffi fails to locate libraries. |
|
|
| msg310594 - (view) |
Author: STINNER Victor (vstinner) * |
Date: 2018-01-24 13:37 |
| I propose to leave MACHDEP unchanged: keep MACHDEP="linux". Can it create inconsistencies? Is it an issue in practice? My short term goal is to use sys.platform == 'android' has the reference test to check if we are running on Android. |
|
|
| msg318102 - (view) |
Author: STINNER Victor (vstinner) * |
Date: 2018-05-29 21:47 |
| Sadly, I failed to find anyone to help me on this issue and I lost interest on supporting Android, so I just close the issue. Sorry. |
|
|
| History |
|
|
|
| Date |
User |
Action |
Args |
| 2022-04-11 14:58:57 |
admin |
set |
github: 76818 |
| 2018-05-29 21:47:52 |
vstinner |
set |
status: open -> closedresolution: out of datemessages: + stage: patch review -> resolved |
| 2018-01-31 04:24:57 |
xgdomingo |
set |
nosy: + xgdomingo |
| 2018-01-25 12:38:38 |
xdegaye |
set |
nosy: - xdegaye |
| 2018-01-24 13:37:33 |
vstinner |
set |
messages: + |
| 2018-01-24 00:44:12 |
vstinner |
set |
nosy: + yan12125 |
| 2018-01-24 00:42:58 |
vstinner |
set |
messages: + |
| 2018-01-24 00:38:05 |
vstinner |
set |
nosy: + doko |
| 2018-01-24 00:12:46 |
vstinner |
set |
nosy: + brett.cannon |
| 2018-01-24 00:12:39 |
vstinner |
set |
messages: + |
| 2018-01-23 23:55:21 |
vstinner |
set |
pull_requests: + <pull%5Frequest5134> |
| 2018-01-23 21:08:03 |
vstinner |
set |
keywords: + patchstage: patch reviewpull_requests: + <pull%5Frequest5133> |
| 2018-01-23 19:39:08 |
pmpp |
set |
nosy: + pmpp |
| 2018-01-23 17:38:53 |
vstinner |
set |
messages: + title: Android: set sys.platform and os.name to android -> Android: set sys.platform to android |
| 2018-01-23 17:16:49 |
vstinner |
create |
|