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 2802

Summary: [patch] Fix android build compiling in wrong filesystem implementation
Product: SDL Reporter: Jonas Kulla <ancurio_bugzilla>
Component: buildAssignee: Gabriel Jacobo <gabomdq>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2 CC: gabomdq, philipp.wiesemann
Version: HG 2.1   
Hardware: All   
OS: Android (All)   
Attachments: patch
patch

Description Jonas Kulla 2014-12-05 13:18:40 UTC 
Created attachment 1953 [details]
patch

The configure script didn't differentiate between Linux and Android,  unconditionally compiling in the unix implementation of SDL_sysfilesystem.c.

'm probably one of the very few people building SDL for android using classic configure + standalone toolchain, so this has gone undetected all along.
Comment 1 Jonas Kulla 2014-12-05 13:30:29 UTC 
Created attachment 1954 [details]
patch

Forgot that you guys keep the generated configure script in revision too.
Comment 2 Philipp Wiesemann 2015-01-26 21:42:59 UTC 
Thank you for the patch. It was applied here:
https://hg.libsdl.org/SDL/rev/541f12d1edcd

I missed that it broke the configure script for FreeBSD (neither Android nor Linux but needs a filesystem implementation). This was fixed here:
https://hg.libsdl.org/SDL/rev/3ad27ca380f3
Comment 3 Jonas Kulla 2015-01-28 06:36:10 UTC 
Ah, my bad, I should have seen that one coming. Thanks for applying and fixing it!