X Tutup
Skip to content

SCons: Fix Windows silence_msvc logfile encoding#110691

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
Ruw-Van:fix-windows-logfile-encoding
Sep 30, 2025
Merged

SCons: Fix Windows silence_msvc logfile encoding#110691
Repiteo merged 1 commit intogodotengine:masterfrom
Ruw-Van:fix-windows-logfile-encoding

Conversation

@Ruw-Van
Copy link
Contributor

@Ruw-Van Ruw-Van commented Sep 19, 2025

When building Godot's source code in a Japanese-language Visual Studio 2022 environment, the build failed due to differences in character encoding between CP932 and Unicode.
This commit resolves the issue, allowing the build to complete successfully.

Fix UnicodeEncodeError on cp932 environments by adding errors="replace" to log file open
@Repiteo
Copy link
Contributor

Repiteo commented Sep 19, 2025

Could you squash your commits? See our pull request guidelines for more information

@Repiteo Repiteo added this to the 4.x milestone Sep 19, 2025
@Ruw-Van Ruw-Van force-pushed the fix-windows-logfile-encoding branch from 85388e0 to b23b0dd Compare September 21, 2025 13:09
@Ruw-Van
Copy link
Contributor Author

Ruw-Van commented Sep 21, 2025

Could you squash your commits? See our pull request guidelines for more information

Thank you for pointing that out.
It seems I had incorporated the Master branch changes without realizing it.
I've now limited it to only the differences from master, so please check again.

Please do so.

@Calinou Calinou added platform:windows cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release labels Sep 24, 2025
@Repiteo Repiteo modified the milestones: 4.x, 4.6 Sep 27, 2025
@Repiteo Repiteo merged commit c6189a8 into godotengine:master Sep 30, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Sep 30, 2025

Thanks! Congratulations on your first merged contribution! 🎉

@Ruw-Van Ruw-Van deleted the fix-windows-logfile-encoding branch October 1, 2025 04:47
@akien-mga akien-mga changed the title Fix windows logfile encoding SCons: Fix Windows silence_msvc logfile encoding Oct 8, 2025
@akien-mga
Copy link
Member

Cherry-picked for 4.5.1.

@akien-mga akien-mga removed the cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release label Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug cherrypick:4.4 Considered for cherry-picking into a future 4.4.x release platform:windows topic:buildsystem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup