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

Allow passing CMAKE_MSVC_RUNTIME_LIBRARY #1960

Merged
merged 1 commit into from
Jun 12, 2024
Merged

Allow passing CMAKE_MSVC_RUNTIME_LIBRARY #1960

merged 1 commit into from
Jun 12, 2024

Conversation

es3n1n
Copy link
Contributor

@es3n1n es3n1n commented Jun 4, 2024

This PR adds support for configuring the MSVC runtime library type using not only CMAKE_C_FLAGS, but also CMAKE_MSVC_RUNTIME_LIBRARY. If a conflict occurs when both CMAKE_MSVC_RUNTIME_LIBRARY and CMAKE_C_FLAGS are set, a fatal error will be raised, as we agreed in #1958.

Both cpp (cmake 3.28.1, MSVC 17.0) and rust (cargo 1.72.0) worked fine on my machine; I honestly don't think this change could break anything.

@wtdcode
Copy link
Member

wtdcode commented Jun 4, 2024

Looks good, waiting for CI

@wtdcode
Copy link
Member

wtdcode commented Jun 6, 2024

Oh wait, you need to rebase to dev branch.

@es3n1n es3n1n changed the base branch from master to dev June 6, 2024 14:03
@es3n1n
Copy link
Contributor Author

es3n1n commented Jun 6, 2024

Oh wait, you need to rebase to dev branch.

Oops, my bad, sorry. I rebased my master to dev and changed the base branch; now, we should be good.

@wtdcode wtdcode merged commit ddd6b7c into unicorn-engine:dev Jun 12, 2024
31 of 32 checks passed
@wtdcode
Copy link
Member

wtdcode commented Jun 12, 2024

Thanks and welcome!

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