From ebffe299ce3b526f0c2d6bd06b90362ef931a583 Mon Sep 17 00:00:00 2001
From: Mitchell Hashimoto <m@mitchellh.com>
Date: Sun, 9 Mar 2025 18:14:53 -0500
Subject: [PATCH] macOS: only set LANGUAGE for app bundle, do not inherit in
 termio env

Fixes #6633

For macOS, we set LANGUAGE to the priority list of preferred languages
for the app bundle, using the GNU gettext priority list format (colon
separated list of language codes).

This previously was inherited by the termio env. At first, this was by
design, but this has inherent flaws. Namely, the priority list format is
a GNU gettext specific format, and programs that use alternate gettext
implementations (like musl or Python) do not understand it and actually
do the wrong thing (not their fault!).

This change removes the inheritance of LANGUAGE in the termio env. To
make it extra safe, we only do set and unset LANGUAGE when we know we
launch from an app bundle. That was always the desired behavior but this
makes it more explicit.
---
 src/apprt/embedded.zig |  7 +++++++
 src/os/locale.zig      | 34 +++++++++++++++++++---------------
 2 files changed, 26 insertions(+), 15 deletions(-)

diff --git a/src/apprt/embedded.zig b/src/apprt/embedded.zig
index 18674bc38..f7f67b270 100644
--- a/src/apprt/embedded.zig
+++ b/src/apprt/embedded.zig
@@ -1039,6 +1039,13 @@ pub const Surface = struct {
 
             // Remove this so that running `ghostty` within Ghostty works.
             env.remove("GHOSTTY_MAC_APP");
+
+            // If we were launched from the desktop then we want to
+            // remove the LANGUAGE env var so that we don't inherit
+            // our translation settings for Ghostty. If we aren't from
+            // the desktop then we didn't set our LANGUAGE var so we
+            // don't need to remove it.
+            if (internal_os.launchedFromDesktop()) env.remove("LANGUAGE");
         }
 
         return env;
diff --git a/src/os/locale.zig b/src/os/locale.zig
index 164a2e6da..f046448ca 100644
--- a/src/os/locale.zig
+++ b/src/os/locale.zig
@@ -108,22 +108,26 @@ fn setLangFromCocoa() void {
     }
 
     // Get our preferred languages and set that to the LANGUAGE
-    // env var in case our language differs from our locale.
-    var buf: [1024]u8 = undefined;
-    if (preferredLanguageFromCocoa(&buf, NSLocale)) |pref_| {
-        if (pref_) |pref| {
-            log.debug(
-                "setting LANGUAGE from preferred languages value={s}",
-                .{pref},
-            );
+    // env var in case our language differs from our locale. We only
+    // do this when the app is launched from the desktop because then
+    // we're in an app bundle and we are expected to read from our
+    // Bundle's preferred languages.
+    if (internal_os.launchedFromDesktop()) language: {
+        var buf: [1024]u8 = undefined;
+        const pref_ = preferredLanguageFromCocoa(
+            &buf,
+            NSLocale,
+        ) catch |err| {
+            log.warn("error getting preferred languages. err={}", .{err});
+            break :language;
+        };
 
-            // TODO: Disabled until we can figure out why this is causing
-            // invalid translations:
-            // https://github.com/ghostty-org/ghostty/discussions/6633
-            // _ = internal_os.setenv("LANGUAGE", pref);
-        }
-    } else |err| {
-        log.warn("error getting preferred languages. err={}", .{err});
+        const pref = pref_ orelse break :language;
+        log.debug(
+            "setting LANGUAGE from preferred languages value={s}",
+            .{pref},
+        );
+        _ = internal_os.setenv("LANGUAGE", pref);
     }
 }