New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
android: SYS_getdents64 does not need to be defined on android API 21 #71049
Comments
Revert the changeset commited at issue bpo-20307 as the compilation does not fail anymore on android API level 21. Patch attached. |
@Shiz: Can we settle on API level 21 or is there any reason to leave this in? |
I have no problem just removing the #ifdef as Android API 21 is now old enough (Lollipop / 5.0) that anyone building Python 3.6 for use on Android is probably fine with it. If there is a #define that can be used to test the android api level at compile time, adding that to the #if is another approach and would keep people who are trying to run something on an older version happy. |
New patch taking into account Gregory's comments. |
The patch is simpler since android/api-level.h is now included in pyport.h by changeset 09af54099973. |
posixmodule_3.patch looks good to me. Gregory has already approved the approach, so I think you can go ahead and commit this. |
Stefan Krah: "posixmodule_3.patch looks good to me. Gregory has already approved the approach, so I think you can go ahead and commit this" I concur with Stefan: LGTM. |
New changeset 4ef3a93e1be2 by Xavier de Gaye in branch 'default': |
Any plans to fix this in 3.5 as well? |
cschramm: AFAIK only Python 3.6+ has experimental Android support. 3.5 or below are not supported. |
Sorry, AFAIK there is no plan to retrofit the Android changes to 3.5. |
Well, the 3.5 code checks __ANDROID__ as well and works pretty fine on Android, but if it's not supposed to be supported, we'll have to upgrade to 3.6 then. Thanks for your work! :) |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: