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 2529

Summary: configure.in includes windows code on non-Windows platforms
Product: SDL Reporter: binarycrusader
Component: buildAssignee: Sam Lantinga <slouken>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2 CC: gabomdq
Version: HG 2.1   
Hardware: x86   
OS: Solaris   

Description binarycrusader 2014-05-09 06:33:36 UTC 
I noticed that when running a build on Solaris (after checking out a fresh copy from mercurial repo) that builds are failing due to SDL_d3dmath.[ch] being included.

Digging into configure.in, the problem seems to stem from lines 337-338 that includes all sources in src/render/* in the build.

It seems like the sources list here should be more selective, as was done with src/filesystem/*.c on line 336.

In fact, I'm confused as to how most of the patterns are valid here as it seems like they would always include all code for all platforms by default, but perhaps that's just my weak autoconf-fu.
Comment 1 binarycrusader 2014-05-09 06:35:03 UTC 
In my particular case, builds are failing because SDL_d3dmath.c is using anonymous struct / union features only found in C11 and in C++ and my compiler is a strict C99 compiler.

But in an off-list email to Sam, it doesn't look like this code is supposed to be built on non-Windows platforms at all.
Comment 2 Gabriel Jacobo 2014-05-10 19:24:47 UTC 
This should be fixed here: https://hg.libsdl.org/SDL/rev/3d5ef7548688

Thanks!