From e1c02b0f0892f48f71855759d8b117f0fc2ff7b4 Mon Sep 17 00:00:00 2001
From: Simon McVittie <smcv@collabora.com>
Date: Fri, 22 Nov 2024 17:30:33 +0000
Subject: [PATCH] launcher-service: Remove obsolete --directory='' from hint

Inheriting the current working directory for the game's s-r-l-s has
been the default when injecting debugging commands into game containers
since mid 2022.

When communicating with `s-r-l-s --alongside-steam` or Flatpak, the
service's current working directory is uninteresting, and trying to
preserve the client's current working directory is usually a better
fit anyway.

So we can reduce visual noise here.

Signed-off-by: Simon McVittie <smcv@collabora.com>
---
 bin/launcher-service.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/bin/launcher-service.c b/bin/launcher-service.c
index f5ecf99eb..da43c4c2e 100644
--- a/bin/launcher-service.c
+++ b/bin/launcher-service.c
@@ -986,12 +986,6 @@ pv_launcher_server_finish_startup (PvLauncherServer *self,
           g_printerr ("\t(launcher address here) \\\n");
         }
 
-      /* --directory='' means keep using the game's working directory.
-       * TODO: This is the default for newer versions of
-       * s-r-launch-client. When suitable versions get into stable releases
-       * of Steam and the various container runtimes, we can reduce
-       * visual noise by removing this option from the hint. */
-      g_printerr ("\t--directory='' \\\n");
       g_printerr ("\t-- \\\n");
       g_printerr ("\tbash\n");
       g_printerr ("\n");
-- 
GitLab