From a4b949b0caacc3394169f4018a5399f564d9d3ea Mon Sep 17 00:00:00 2001 From: Johannes Altmanninger Date: Fri, 28 Nov 2025 10:31:33 +0100 Subject: [PATCH] Remove uname= prefix from query-os-name Discussion with terminal authors indicates a slight preference for removing the prefix. We don't need it at this point, since we only use it to detect macOS clients. --- doc_src/terminal-compatibility.rst | 2 +- share/functions/fish_in_macos_terminal.fish | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/doc_src/terminal-compatibility.rst b/doc_src/terminal-compatibility.rst index 3882949ff..d0354f173 100644 --- a/doc_src/terminal-compatibility.rst +++ b/doc_src/terminal-compatibility.rst @@ -273,7 +273,7 @@ Optional Commands The response's second parameter is ignored. * ``query-os-name`` (for :ref:`status terminal-os `) - Terminals running on Unix should respond with the hex encoding of ``uname=$(uname)`` as second parameter. + Terminals running on Unix should respond with the hex encoding of ``$(uname -s)`` as second parameter. .. _term-compat-dcs-gnu-screen: diff --git a/share/functions/fish_in_macos_terminal.fish b/share/functions/fish_in_macos_terminal.fish index 2668a0958..02fffa47a 100644 --- a/share/functions/fish_in_macos_terminal.fish +++ b/share/functions/fish_in_macos_terminal.fish @@ -1,3 +1,3 @@ function fish_in_macos_terminal - test "$(status terminal-os || echo uname="$(__fish_uname)")" = uname=Darwin + test "$(status terminal-os || echo "$(__fish_uname)")" = Darwin end