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

NOMINMAX can be defined already on Windows #210

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

VA-GS
Copy link
Contributor

@VA-GS VA-GS commented Oct 3, 2024

Part of the changes discussed here.

I am building on Windows with MSVC, MSYS2 gcc and clang. I will submit a pull request for https://github.com/mas-bandwidth/netcode too. For the github actions, I will open an issue instead, there are a couple of issues with it.

@gafferongames
Copy link
Contributor

Windows build doesn't seem to like this with Visual Studio...

@gafferongames
Copy link
Contributor

I'm happy to consider this change, but it needs to be modified so it passes tests.

@VA-GS
Copy link
Contributor Author

VA-GS commented Oct 7, 2024

Sorry, I did not mean to add the ci.yml changes here, only 1 commit. Should be better now.

@gafferongames gafferongames merged commit a163cf2 into mas-bandwidth:main Oct 7, 2024
12 checks passed
@gafferongames
Copy link
Contributor

Merged. Thanks!

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