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

Required changes to build for Windows #676

Closed
wants to merge 1 commit into from

Conversation

aarononeal
Copy link

I had to add these to build for Windows. Anyone else find that these are required?

@cmitu
Copy link

cmitu commented Aug 5, 2020

What is your build environment ? The current source works fine with VS 2017/2015.

@aarononeal
Copy link
Author

aarononeal commented Aug 5, 2020

  • VS 2019
  • SDL 2.0.12
  • LibVLC 3.0.11

@cmitu
Copy link

cmitu commented Aug 6, 2020

I've trying building using VS 2019 (Community) and there's no error during compilation. Linking fails because of SAFESEH being enabled, but after disabling it in the linker options, the build succeeds.
Are you using clang for compilation ?

@retroNUC
Copy link

retroNUC commented Jun 29, 2021

An update from someone who's recently been building Win32 via VS2019 / MSVC Build Tools - The only required change for compilation is adding #include <string> to ResourceManager.h.

@tomaz82 tomaz82 mentioned this pull request Jan 19, 2022
@tomaz82
Copy link
Collaborator

tomaz82 commented Jan 25, 2022

Implemented by #774

@tomaz82 tomaz82 closed this Jan 25, 2022
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.

None yet

4 participants