We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 3823 - Android debug build problem with library name suffix when using cmake
Summary: Android debug build problem with library name suffix when using cmake
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: build (show other bugs)
Version: HG 2.0
Hardware: All Android (All)
: P2 normal
Assignee: Sam Lantinga
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-09-13 12:59 UTC by Olli Kallioinen
Modified: 2018-02-25 18:32 UTC (History)
0 users

See Also:


Attachments
A fix that disables adding a suffix to debug library name on android (cmake) (949 bytes, patch)
2017-09-13 13:01 UTC, Olli Kallioinen
Details | Diff
Don't add a suffix to debug library name on android (cmake) (2.07 KB, patch)
2018-02-25 17:35 UTC, Olli Kallioinen
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Olli Kallioinen 2017-09-13 12:59:51 UTC
Cmake currently adds a 'd' to the end of the output library name in debug builds. This is a problem on android as the native libraries are loaded explicitly by name.

I made a fix to just disable this suffix on android. Maybe another option would be to make the library loading code somehow more intelligent, but I'm not sure if that's really useful.
Comment 1 Olli Kallioinen 2017-09-13 13:01:37 UTC
Created attachment 2930 [details]
A fix that disables adding a suffix to debug library name on android (cmake)
Comment 2 Olli Kallioinen 2018-02-25 17:35:31 UTC
Created attachment 3182 [details]
Don't add a suffix to debug library name on android (cmake)

Added an up-to-date patch that also sets the debug suffix per target so that the SDL cmake file does not mess with global cmake properties (similarly to the patch in https://bugzilla.libsdl.org/show_bug.cgi?id=3918)
Comment 3 Sam Lantinga 2018-02-25 18:32:39 UTC
Added, thanks!
https://hg.libsdl.org/SDL/rev/43dbca2813e1