X Tutup
Skip to content

Set language encoding flag when using Pack Project as ZIP#110387

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
timothyqiu:project-zip-utf8
Sep 22, 2025
Merged

Set language encoding flag when using Pack Project as ZIP#110387
Repiteo merged 1 commit intogodotengine:masterfrom
timothyqiu:project-zip-utf8

Conversation

@timothyqiu
Copy link
Member

Similar to #78732

Currently, the zip archive produced by the "Pack Project as ZIP" command does not have its language encoding flag set. Therefore, the filenames inside might not be recognized as UTF-8 on Windows, causing garbled filenames in extracted files.

@timothyqiu timothyqiu added this to the 4.6 milestone Sep 10, 2025
@timothyqiu timothyqiu requested a review from a team September 10, 2025 13:05
@timothyqiu timothyqiu added bug topic:editor cherrypick:4.5 Considered for cherry-picking into a future 4.5.x release labels Sep 10, 2025
Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good to me.

PS: I'm curious as to why this code doesn't use ZIPPacker instead. In the long run, it would make sense to avoid low-level minizip usage unless absolutely necessary.

@Repiteo Repiteo merged commit f553875 into godotengine:master Sep 22, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Sep 22, 2025

Thanks!

@timothyqiu timothyqiu deleted the project-zip-utf8 branch September 23, 2025 00:43
@akien-mga
Copy link
Member

Cherry-picked for 4.5.2.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

X Tutup