Document usage of gnustl_shared with Android toolchain
ClosedPublic

Authored by kossebau on Dec 19 2016, 2:30 PM.

Diff Detail

Repository
R240 Extra CMake Modules
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau updated this revision to Diff 9167.Dec 19 2016, 2:30 PM
kossebau retitled this revision from to Document usage of gnustl_shared with Android toolchain.
kossebau updated this object.
kossebau added reviewers: Frameworks, apol, cordlandwehr.

http://doc.qt.io/qbs/android-ndk-module.html allows to control which helper runtime to use, so hardcoding gnustl_shared in the ECM toolchain could have been done for a certain reason or perhaps just was done once to get started, but never revisited?

apol added inline comments.Dec 19 2016, 2:50 PM
toolchain/Android.cmake
160

when I started working on Android it was the only one available and it's the one we test things on on GNU/Linux, so I'd say it's safer to stay on gnustl unless there's a very good reason to move away from it.

164

There's not really a chance, if it's not there it's an error. I'd remove this comment altogether.

kossebau updated this revision to Diff 9168.Dec 19 2016, 3:12 PM
  • Added explanation for hardcoding gnustl_shared for now
  • Removed comment as proposed
apol accepted this revision.Dec 19 2016, 6:31 PM
apol edited edge metadata.
This revision is now accepted and ready to land.Dec 19 2016, 6:31 PM
This revision was automatically updated to reflect the committed changes.
kossebau marked an inline comment as done.