Differential D19016

Update Android toolchain files to reality
ClosedPublic
Actions

Authored by vkrause on Feb 14 2019, 5:30 PM.

Details

Reviewers
apol
Commits
R240:81374904954b: Update Android toolchain files to reality
Summary
  • NDK r18 does not ship GCC anymore, it's only clang now
  • same for STL, there's only libc++ there
  • API level 14 is not supported either anymore, the minimum supported by the current NDK is 16
  • The minimum supported API level by KF5 right now is 21 though, so use that as the default

Diff Detail

Repository
R240 Extra CMake Modules
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 8306
Build 8324: arc lint + arc unit
vkrause created this revision.Feb 14 2019, 5:30 PM
Restricted Application added projects: Frameworks, Build System. · View Herald TranscriptFeb 14 2019, 5:30 PM
Restricted Application added subscribers: kde-buildsystem, kde-frameworks-devel. · View Herald Transcript
vkrause requested review of this revision.Feb 14 2019, 5:30 PM
apol accepted this revision.Feb 15 2019, 12:41 PM
apol added a subscriber: apol.

Thanks!

This revision is now accepted and ready to land.Feb 15 2019, 12:41 PM
Closed by commit R240:81374904954b: Update Android toolchain files to reality (authored by vkrause). · Explain WhyFeb 15 2019, 1:05 PM
This revision was automatically updated to reflect the committed changes.
jjazeix added a subscriber: jjazeix.Feb 25 2019, 9:29 PM
jjazeix added inline comments.
toolchain/Android.cmake
150

Sorry, I'm a bit late, but would it be possible to put 16 as minimum here? Some projects (GCompris at least) do not use KF5 and would like to target lower API levels.
I can do the diff/commit to change it if necessary

apol added a comment.Feb 26 2019, 2:54 PM

Applications can still override it, note it's a default.
Also note this: https://developer.android.com/distribute/best-practices/develop/target-sdk

jjazeix added a comment.Feb 26 2019, 3:57 PM
In D19016#420246, @apol wrote:

Applications can still override it, note it's a default.
Also note this: https://developer.android.com/distribute/best-practices/develop/target-sdk

Ok, good for me :).
Thanks for the link!

Revision Contents
Changeset List

PathPackages
Mtoolchain/Android.cmake (6 lines)

Diff 51701

View Options

toolchain/Android.cmake

Loading...