From 3242ce340621839d03ecf34bb3ed20d7c6f3cbb3 Mon Sep 17 00:00:00 2001 From: Ophestra Umiker Date: Mon, 9 Sep 2024 00:35:16 +0900 Subject: app: treat display server variable unset as fatal This is yet another remnant of Ego, as Ego unconditionally shares these resources and the absence of them are ignored and warned about in verbose logging. In our case they are individually opt-in so silently dropping them while the enablement is still set makes very little sense. Signed-off-by: Ophestra Umiker --- internal/app/wayland.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'internal/app/wayland.go') diff --git a/internal/app/wayland.go b/internal/app/wayland.go index 1cabdbdb..3b0be85f 100644 --- a/internal/app/wayland.go +++ b/internal/app/wayland.go @@ -20,9 +20,7 @@ func (a *App) ShareWayland() { // ensure Wayland socket ACL (e.g. `/run/user/%d/wayland-%d`) if w, ok := os.LookupEnv(waylandDisplay); !ok { - if system.V.Verbose { - fmt.Println("Wayland: WAYLAND_DISPLAY not set, skipping") - } + state.Fatal("Wayland: WAYLAND_DISPLAY not set") } else { // add environment variable for new process wp := path.Join(system.V.Runtime, w) -- cgit v1.3.1