Skip to content

Add support for minimum Android API. #1197

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

Closed
wants to merge 1 commit into from
Closed

Add support for minimum Android API. #1197

wants to merge 1 commit into from

Conversation

tito
Copy link
Member

@tito tito commented Dec 15, 2017

This allows the compilation to use a different version for compiling while targetting a higher API version.

Typically, you may need to use API 23 for Java, but want to run on devices with API 19. If you compile with API 23, when running on older device (let's say 4.4.2), you'll end up with a symbol not found "srand", "signal", etc.

It is important to set the Android API minimum version when compiling in order to avoid theses symbol issues.

CF: https://stackoverflow.com/a/41079462/69877

This allows the compilation to use a different version for compiling
while targetting a higher API version.

Typically, you may need to use API 23 for Java, but want to run on API
19. If you compile with API 23, when running on older device (let's say
4.4.2), you'll end up with a symbol not found "srand", "signal", etc.

It is important to set the Android API minimum version when compiling in
order to avoid theses symbol issues.

CF: https://stackoverflow.com/a/41079462/69877
@QuadTriangle
Copy link
Contributor

I also had to change this line to successfully compile python2

' -D__ANDROID_API__={}'.format(self.ctx._android_api),

still, getting 'srand' missing error from 'libSDL2.so'

@inclement
Copy link
Member

@QuadTriangle I just did a quick test and it worked okay, do you know what needs to happen to duplicate your problem?

akinwale added a commit to lbryio/lbry-android that referenced this pull request Apr 11, 2018
… API support. Minimum Android API set to 21.
akinwale added a commit to lbryio/lbry-android that referenced this pull request Apr 11, 2018
@AndreMiras
Copy link
Member

Hi @tito and @inclement was this covered during the recent refactoring in #1412 ? If so feel free to close, otherwise maybe we want to rebase and fix the conflicts.

@inclement
Copy link
Member

Yep, the python3 changes implement the same thing (plus some fixes beyond this).
I don't think I ever saw this PR :/

@inclement inclement closed this Nov 11, 2018
@tito tito deleted the feature-minapi branch January 30, 2019 16:28
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