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

Change Windows.h to windows.h to help with cross-compilation. #123

Merged
merged 1 commit into from
Apr 23, 2024

Conversation

HolyBlackCat
Copy link
Contributor

Hello!

Currently this library can't be cross-compiled to Windows from e.g. Linux, because the header names are case-sensitive there, and you're using #include <Windows.h>, while mingw has this file named windows.h (lowercase).

I've changed it to lowercase everywhere.

@dougbinks
Copy link
Owner

The Windows.h header supplied by Microsoft has an upper case first letter (at least for a couple of decades), so the real issue is that the windows.h header used for cross compilation has an incorrect first letter. It may be worth filing an issue with the cross compilation toolkit.

As for this workaround, Windows NTFS can be case-sensitive, but since this is per-folder I believe it's unlikely that the SDK folders will have this set, and thus it should be safe to use this workaround. I will test and merge when I get some time.

@dougbinks dougbinks changed the base branch from master to dev April 23, 2024 14:33
@dougbinks dougbinks merged commit 086ead6 into dougbinks:dev Apr 23, 2024
4 checks passed
@dougbinks
Copy link
Owner

Merged, many thanks for the PR!

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

Successfully merging this pull request may close these issues.

2 participants