Skip to content

fix(oot): a Brazilian Portuguese Mac or app language plays Ocarina of Time in Brazilian Portuguese - #2161

Open
victormlourenco wants to merge 1 commit into
ly05010419:mainfrom
victormlourenco:fix/oot-pt-br
Open

victormlourenco wants to merge 1 commit into
ly05010419:mainfrom
victormlourenco:fix/oot-pt-br

Conversation

@victormlourenco

Copy link
Copy Markdown

Problem

With the Mac's language, or the app's own language (defaults write org.macnativegames.oot AppleLanguages -array pt-BR), set to Brazilian Portuguese, Ocarina of Time chose European Portuguese (pt). desired_language in games/oot/runtime/services.c passes brazil=0 to game_language() for every game except Tears of the Kingdom 1.4.3 and Breath of the Wild 1.9.0, so pt-BR became pt.

Fix

The v131072 RomFS has the official Brazilian Portuguese text: assets/romfs-v131072.tsv.gz lists 1,719 files under USpt, next to USen (1,747) and USes (1,715). GAME_OOT now takes brazil=1 too, and the comment above the function names it. The change is in Ocarina of Time's own runtime/services.c only; no other game is affected.

Testing

  • Played on a Mac (Apple M4 Max, macOS 27.2): Ocarina of Time v131072, runtime updated with update_runtime.py, AppleLanguages of org.macnativegames.oot set to pt-BR. The game log has [totk] language: pt-BR (TOTK_LANG=- TOTK_LOCALE=pt-BR), and the game started in Brazilian Portuguese.
  • No Python changed. The C change is one preprocessor condition plus a comment, and the runtime builds with no errors.

… Time in Brazilian Portuguese

desired_language passed brazil=0 for Ocarina of Time, so a pt-BR locale (the
Mac's, or the app's own AppleLanguages) mapped to European "pt". The game's
v131072 RomFS has the official Brazilian Portuguese text (1,719 files under
USpt, next to USen and USes), as Tears of the Kingdom 1.4.3 and Breath of the
Wild 1.9.0 do, so GAME_OOT now takes brazil=1 as well.
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.

1 participant