From e81f525101ab7f83d917ca9d8dd0946ec7ca5de5 Mon Sep 17 00:00:00 2001 From: Mitchell Hashimoto Date: Wed, 8 May 2024 21:04:50 -0700 Subject: [PATCH] apprt/gtk: set GDK_DEBUG --- src/apprt/gtk/App.zig | 8 ++++++++ src/main_ghostty.zig | 4 ---- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/src/apprt/gtk/App.zig b/src/apprt/gtk/App.zig index 6da314c56..7e7c50900 100644 --- a/src/apprt/gtk/App.zig +++ b/src/apprt/gtk/App.zig @@ -64,6 +64,14 @@ x11_xkb: ?x11.Xkb = null, pub fn init(core_app: *CoreApp, opts: Options) !App { _ = opts; + // We need to export GDK_DEBUG to run on Wayland after GTK 4.14. + // Older versions of GTK do not support these values so it is safe + // to always set this. Forwards versions are uncertain so we'll have to + // reassess... + // + // Upstream issue: https://gitlab.gnome.org/GNOME/gtk/-/issues/6589 + _ = internal_os.setenv("GDK_DEBUG", "opengl,gl-disable-gles"); + // Load our configuration var config = try Config.load(core_app.alloc); errdefer config.deinit(); diff --git a/src/main_ghostty.zig b/src/main_ghostty.zig index 515b3e1fb..ff38b800d 100644 --- a/src/main_ghostty.zig +++ b/src/main_ghostty.zig @@ -239,10 +239,6 @@ pub const GlobalState = struct { self.logging = .{ .disabled = {} }; } - // We need to export GDK_DEBUG to run on wayland after gtk 4.14 - // Upstream issue: https://gitlab.gnome.org/GNOME/gtk/-/issues/6589 - if (builtin.target.os.tag == .linux) _ = internal_os.setenv("GDK_DEBUG", "opengl,gl-disable-gles"); - // I don't love the env var name but I don't have it in my heart // to parse CLI args 3 times (once for actions, once for config, // maybe once for logging) so for now this is an easy way to do