Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[WIP] bpo-32654: Fixes for Android API 19#5305

Closed
vstinner wants to merge 10 commits into
python:masterpython/cpython:masterfrom
vstinner:android_api19Copy head branch name to clipboard
Closed

[WIP] bpo-32654: Fixes for Android API 19#5305
vstinner wants to merge 10 commits into
python:masterpython/cpython:masterfrom
vstinner:android_api19Copy head branch name to clipboard

Conversation

@vstinner

@vstinner vstinner commented Jan 24, 2018

Copy link
Copy Markdown
Member
  • Declare mmap() prototype on API < 21
  • Avoid setlocale(LC_ALL, NULL) in Py_Main() on API < 20
  • pystrtod.c: avoid localeconv() on API < 21
  • _Py_GetLocaleconvNumeric(): avoid localeconv() on API < 20, use
    hardcoded values.

https://bugs.python.org/issue32654

* Declare mmap() prototype on API < 21
* Avoid setlocale(LC_ALL, NULL) in Py_Main() on API < 20
* pystrtod.c: avoid localeconv() on API < 21
* _Py_GetLocaleconvNumeric(): avoid localeconv() on API < 20, use
  hardcoded values.
@vstinner vstinner changed the title bpo-32654: Fixes for Android API 19 [WIP] bpo-32654: Fixes for Android API 19 Jan 24, 2018
@vstinner

Copy link
Copy Markdown
Member Author

@yan12125: Would you mind to review and/or test this PR?

I'm not sure that we should support Android API 19, see: https://bugs.python.org/issue32654#msg310682

@yan12125

Copy link
Copy Markdown

@yan12125: Would you mind to review and/or test this PR?

Sorry, I may not be able to for now. I don't have a working android-19 device, and I don't think testing on an emulator can give trustworthy results. In my experience there are lots of bugs occur on emulators only.

@vstinner

Copy link
Copy Markdown
Member Author

Sorry, I may not be able to for now. I don't have a working android-19 device, and I don't think testing on an emulator can give trustworthy results. In my experience there are lots of bugs occur on emulators only.

The question is not if this PR is enough to fix all API 19 issues, just if it doesn't make things worse :-D

@yan12125

Copy link
Copy Markdown

Oh, of course it won't be worse than crash immediately after startup :)

I can have a check on emulators, possibly this weekend.

Comment thread Python/pystrtod.c Outdated
fail_pos = NULL;

locale_data = localeconv();
#if defined(__ANDROID_API__) and __ANDROID_API__ < 21

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

and => &&

It's not Python :)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops :-) Fixed.

@vstinner vstinner closed this Feb 1, 2018
@vstinner vstinner deleted the android_api19 branch May 29, 2018 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.