Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CMAKE: STDC_HEADERS is not defined except for msvc [patch] #2042

Closed
SDLBugzilla opened this issue Feb 11, 2021 · 0 comments
Closed

CMAKE: STDC_HEADERS is not defined except for msvc [patch] #2042

SDLBugzilla opened this issue Feb 11, 2021 · 0 comments

Comments

@SDLBugzilla
Copy link
Collaborator

This bug report was migrated from our old Bugzilla tracker.

These attachments are available in the static archive:

Reported in version: HG 2.1
Reported for operating system, platform: All, All

Comments on the original bug report:

On 2015-12-06 10:29:52 +0000, Martin Gerhardy wrote:

Created attachment 2330
the patch for the cmake file

This leads to all kind of weird issues - but on most platforms to compile errors in SDL_stdinc.h because wchar_t is defined in stddef.h

On 2015-12-29 05:30:17 +0000, Ryan C. Gordon wrote:

This is defined for non-MSVC targets in the else() block:

check_include_files("dlfcn.h;stdint.h;stddef.h;inttypes.h;stdlib.h;strings.h;string.h;float.h" STDC_HEADERS)

What platform was this failing on? I could see this failing for, say, MingW, which I presume doesn't offer dlfcn.h.

--ryan.

On 2015-12-29 06:00:34 +0000, Ryan C. Gordon wrote:

(In reply to Ryan C. Gordon from comment # 1)

What platform was this failing on? I could see this failing for, say, MingW,
which I presume doesn't offer dlfcn.h.

Just discovered from Bug # 3194 that this was probably an Android problem, right? We might just need to special-case that list of headers. I don't think we want to turn on STDC_HEADERS universally.

--ryan.

On 2015-12-29 06:27:57 +0000, Martin Gerhardy wrote:

yes. was an android issue

On 2015-12-31 08:17:58 +0000, Ryan C. Gordon wrote:

(In reply to Martin Gerhardy from comment # 3)

yes. was an android issue

Is it float.h that's missing?

--ryan.

On 2015-12-31 10:14:17 +0000, Martin Gerhardy wrote:

Weird, but i'm not able to reproduce this anymore. I've tried mingw64_32, emscripten and android ndk. Unfortunately I didn't document which version and which platform it was - so I retested all of them. But i've updated emscripten and the ndk in the meanwhile. So I would say, just close this one and if I ever encounter it again, I will document the stuff properly.

On 2016-01-01 06:06:16 +0000, Ryan C. Gordon wrote:

Marking fixed, then...reopen if it pops up again!

--ryan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant