X Tutup
Skip to content

Fix Chinese characters in DBusMessage were not displayed correctly#112629

Merged
Repiteo merged 1 commit intogodotengine:masterfrom
Rindbee:fix-Chinese-characters-were-not-displayed-correctly-in-DBusMessage
Nov 11, 2025
Merged

Fix Chinese characters in DBusMessage were not displayed correctly#112629
Repiteo merged 1 commit intogodotengine:masterfrom
Rindbee:fix-Chinese-characters-were-not-displayed-correctly-in-DBusMessage

Conversation

@Rindbee
Copy link
Contributor

@Rindbee Rindbee commented Nov 11, 2025

On OS with Chinese locale, you might receive the following message when running an editor in i3wm using the --verbose option:

ERROR: Error on D-Bus communication: æªæ	¾å°è¯·æ±ç
                                                         设置
   at: read_setting (platform/linuxbsd/freedesktop_portal_desktop.cpp:144)

After applying this PR, the error message will change to:

ERROR: Failed to read accent-color in org.freedesktop.appearance, due to error on D-Bus communication: 未找到请求的设置
   at: read_setting (platform/linuxbsd/freedesktop_portal_desktop.cpp:144)

On OS with Chinese locale, you might receive the following message
when running an editor using the `--verbose` option:

```
ERROR: Error on D-Bus communication: æªæ	¾å°è¯·æ±ç
                                                         设置
   at: read_setting (platform/linuxbsd/freedesktop_portal_desktop.cpp:144)
```
@Rindbee Rindbee requested a review from a team as a code owner November 11, 2025 00:30
@Rindbee Rindbee changed the title Fix Chinese characters were not displayed correctly in DBusMessage Fix Chinese characters in DBusMessage were not displayed correctly Nov 11, 2025
@bruvzg bruvzg added this to the 4.6 milestone Nov 11, 2025
@Repiteo Repiteo merged commit 4e2adc6 into godotengine:master Nov 11, 2025
20 checks passed
@Repiteo
Copy link
Contributor

Repiteo commented Nov 11, 2025

Thanks!

@Rindbee Rindbee deleted the fix-Chinese-characters-were-not-displayed-correctly-in-DBusMessage branch November 11, 2025 23:24
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.

3 participants

X Tutup