Skip to content

Conversation

@akien-mga
Copy link
Member

  • Fedora 43
  • SCons 4.10.1
  • MinGW 13.0.0 with GCC 15.2.1 and binutils 2.45
  • Emscripten 4.0.20
  • Xcode 26.1.1

- Fedora 43
- SCons 4.10.1
- MinGW 13.0.0 with GCC 15.2.1 and binutils 2.45
- Emscripten 4.0.20
- Xcode 26.1.1
@akien-mga akien-mga marked this pull request as ready for review December 16, 2025 10:32
@akien-mga
Copy link
Member Author

We've been using this in production for a few weeks (4.6 dev6 and beta1 at least), I just forgot to merge this.

@akien-mga akien-mga merged commit a8991fa into main Dec 16, 2025
@akien-mga akien-mga deleted the f43-update-4.6 branch December 16, 2025 10:32
@bruvzg
Copy link
Member

bruvzg commented Dec 16, 2025

MinGW 13.0.0 with GCC 15.2.1 and binutils 2.45

This will likely require changing this two lines:
https://github.com/godotengine/godot-build-scripts/blob/5261d2c468a9ad3dcf643ba2174105402a3a8b4b/build.sh#L186-L187

-  curl -L -o windows_x86_64.zip $base_url-x86_64-gcc-release.zip
-  curl -L -o windows_x86_32.zip $base_url-x86_32-gcc-release.zip
+  curl -L -o windows_x86_64.zip $base_url-x86_64-gcc-13-release.zip
+  curl -L -o windows_x86_32.zip $base_url-x86_32-gcc-13-release.zip

@bruvzg
Copy link
Member

bruvzg commented Dec 16, 2025

We've been using this in production for a few weeks (4.6 dev6 and beta1 at least), I just forgot to merge this.

Or if it's working as is, it might be only backward incompatible and lib built with MinGW 12 will work with MinGW 13. Probably better to change it anyway to match runtime version.

@akien-mga
Copy link
Member Author

Yeah it probably makes sense to change those, I missed that we had two sets of binaries for ANGLE.

akien-mga added a commit to godotengine/godot-build-scripts that referenced this pull request Dec 16, 2025
@akien-mga
Copy link
Member Author

Updated with godotengine/godot-build-scripts@9fb0561.

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