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

Release builds are linking with debug libraries #917

Closed
codecat opened this issue Nov 8, 2022 · 4 comments · Fixed by #932
Closed

Release builds are linking with debug libraries #917

codecat opened this issue Nov 8, 2022 · 4 comments · Fixed by #932
Labels
bug This has been identified as a bug platform:windows topic:buildsystem Related to the buildsystem or CI setup

Comments

@codecat
Copy link
Contributor

codecat commented Nov 8, 2022

Since updating for Godot 4 beta 4 support, users have been reporting an issue where my builds produced using template_release are linked with debug runtime libraries: codecat/godot-tbloader#36

This might be related to #867, but I don't really know enough about SCons to properly debug this. I believe my SCons file is properly configured though, so this might be a bug in godot-cpp?

@akien-mga
Copy link
Member

Was this not a bug in your SConstruct that was fixed by codecat/godot-tbloader@845a29a ?

@codecat
Copy link
Contributor Author

codecat commented Nov 11, 2022

That was an attempt at fixing it which sadly did not work.

@akien-mga
Copy link
Member

I'm not sure that's the source of the problem but you could try to force debug_symbols=no to skip this branch:
https://github.com/godotengine/godot-cpp/blob/master/tools/windows.py#L30-L31
(BTW the target check is wrong @Faless)

We should ensure that debug_symbols only affects generating and emitting debug symbols, like in upstream Godot. We currently don't have flags that make MSVC use /MDd in Godot (it could for dev_build, but debug_symbols should stay something "extra" that generates symbols that can be used or not - i.e. when not shipping the pdb, it should behave like a build without debug symbols).

@akien-mga akien-mga added bug This has been identified as a bug topic:buildsystem Related to the buildsystem or CI setup platform:windows labels Nov 11, 2022
@codecat
Copy link
Contributor Author

codecat commented Nov 11, 2022

Using debug_symbols=no seems to work around this and properly links to the release dll's. 👍 This is a good workaround for me now, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This has been identified as a bug platform:windows topic:buildsystem Related to the buildsystem or CI setup
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants