-
-
Notifications
You must be signed in to change notification settings - Fork 45
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
Issues with enabling BTLS on Windows #66
Comments
I'll have to update the above, it seems like I misunderstood things and I did unnecessarily reverts this morning, this doesn't seem to be the cause of the issue I'm debugging at godotengine/godot#61176. |
If this causes too much trouble, we may as well forget about supporting this in Godot 3.x. It's too much trouble. In 4.0 we don't need to worry about this at all, it will just work. |
So after more testing I found that godotengine/godot#61176 was indeed not related to BTLS at all, but it's a MinGW + GCC + binutils regressions which present in a specific combination of versions of these components, as found in Fedora 35. Using Fedora 34 (like we used to) or Fedora 36 as a baseline works fine. So we could revert my revert commits and re-add btls-shared on Windows builds. I thought that we didn't have it in my previous Fedora 34 based container due to godotengine/build-containers@d8d17c5, but actually I built the Windows container one day before that commit - probably I made that commit to fix the macOS build after the Windows build had succeeded, so they used a slightly different version of godot-mono-builds (which is cloned in the host by the build script). But indeed, since we don't intend to do more work to actually make this useful, it's maybe fine as is. My (limited) understanding is that even though we managed to build btls-shared, this didn't solve the issues that users are having in godotengine/godot#54758. |
So according to godotengine/godot#61364 the BTLS lib on Windows was actually working/useful (CC @zaksnet). So I'll revert my revert so we build it again on Windows. Edit: Done: 2fa04b9 |
OS/device including version:
Windows, cross-compiling from Fedora with MinGW
Issue description:
PR #49 introduced build issues that affect both macOS ARM64 builds and the Windows 64-bit builds (at least cross-compiled from Fedora with MinGW).
See godotengine/build-containers#95 for the macOS ARM64 build.
For several months I had this PR reverted locally in build-containers to solve it (godotengine/build-containers@d8d17c5), and then in #64 and #65 we decided to improve it to make it Windows-only to prevent the macOS ARM64 issue.
And I've used that in 3.5 RC 1 builds based on c865201, but it seems that it's still broken on Windows 64-bit now: godotengine/godot#61176
So for now I've reverted #65 and #49 with 20368d7 and aaee521.
This can be looked into further to figure out how to actually make BTLS support work on Windows with MinGW, and possibly finally fix godotengine/godot#54758 (ref #47 too for context). Until we're sure that it's working, keeping these WIP patches seem to do more harm than good.
The text was updated successfully, but these errors were encountered: