[PR #4860] fix: Set TERM=screen for all non-interactive tmux calls #4460

New issue

Open

opened 2026-02-27 04:55:14 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 04:55:14 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/GameServerManagers/LinuxGSM/pull/4860
Author: @silasjmatson
Created: 12/1/2025
Status: 🔄 Open

Base: develop ← Head: master

📝 Commits (1)

59e6205 fix: Set TERM=screen for all non-interactive tmux calls

📊 Changes

6 files changed (+15 additions, -15 deletions)

View changed files

📝 lgsm/modules/check_status.sh (+1 -1)
📝 lgsm/modules/command_send.sh (+1 -1)
📝 lgsm/modules/command_start.sh (+4 -4)
📝 lgsm/modules/command_stop.sh (+6 -6)
📝 lgsm/modules/info_distro.sh (+1 -1)
📝 lgsm/modules/update_xnt.sh (+2 -2)

📄 Description

Description

When connected to a server via ssh, and the server doesn't have the
proper terminfo, tmux will fail with 'unsupported terminal' errors.

These errors are not surfaced by the scripts, e.g. check_status,
so they result in either silent failures or don't work at all.

e.g. `./mcserver details` on my server showed 0% CPU, 0% Memory,
      and STOPPED for the server status when the server was
      actually running. This was because the tmux calls to get
      this info failed.

      The root cause of this was that I was connected to my server
      via kitty, a terminal emulator that my server did not have
      terminfo for.

This can be resolved via running infocmp -x xterm-kitty | ssh YOUR-SERVER -- tic -x -
and replacing xterm-kitty with whatever is appropriate, or by setting
TERM=xterm-256color in the ssh config, but that is not obvious to do
when presented with silent failures.

This commit sets TERM=screen for all non-interactive tmux calls,
which resolves the issue. Note that the console command does not
set this, as that is an interactive shell, and the user's terminfo
should be passed.

Pastebin log demonstrating the issue with ghostty terminal: https://pastebin.com/JeAVUfk4

Fixes: #4861

Type of change

Bug fix (a change which fixes an issue).

Checklist

PR will not be merged until all steps are complete.

This pull request links to an issue.
This pull request uses the develop branch as its base.
This pull request subject follows the Conventional Commits standard.
This code follows the style guidelines of this project.
I have performed a self-review of my code.
I have checked that this code is commented where required.
I have provided a detailed enough description of this PR.
I have checked if documentation needs updating.

Documentation

If documentation does need updating either update it by creating a PR (preferred) or request a documentation update.

User docs: https://github.com/GameServerManagers/LinuxGSM-Docs
Dev docs: https://github.com/GameServerManagers/LinuxGSM-Dev-Docs

Thank you for your Pull Request!

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/GameServerManagers/LinuxGSM/pull/4860 **Author:** [@silasjmatson](https://github.com/silasjmatson) **Created:** 12/1/2025 **Status:** 🔄 Open **Base:** `develop` ← **Head:** `master` --- ### 📝 Commits (1) - [`59e6205`](https://github.com/GameServerManagers/LinuxGSM/commit/59e62052679fbc8d1c4fe809d237eff3a5b1b9ec) fix: Set TERM=screen for all non-interactive tmux calls ### 📊 Changes **6 files changed** (+15 additions, -15 deletions) <details> <summary>View changed files</summary> 📝 `lgsm/modules/check_status.sh` (+1 -1) 📝 `lgsm/modules/command_send.sh` (+1 -1) 📝 `lgsm/modules/command_start.sh` (+4 -4) 📝 `lgsm/modules/command_stop.sh` (+6 -6) 📝 `lgsm/modules/info_distro.sh` (+1 -1) 📝 `lgsm/modules/update_xnt.sh` (+2 -2) </details> ### 📄 Description # Description When connected to a server via ssh, and the server doesn't have the proper terminfo, tmux will fail with 'unsupported terminal' errors. These errors are not surfaced by the scripts, e.g. check_status, so they result in either silent failures or don't work at all. e.g. `./mcserver details` on my server showed 0% CPU, 0% Memory, and STOPPED for the server status when the server was actually running. This was because the tmux calls to get this info failed. The root cause of this was that I was connected to my server via kitty, a terminal emulator that my server did not have terminfo for. This can be resolved via running `infocmp -x xterm-kitty | ssh YOUR-SERVER -- tic -x -` and replacing `xterm-kitty` with whatever is appropriate, or by setting TERM=xterm-256color in the ssh config, but that is not obvious to do when presented with silent failures. This commit sets `TERM=screen` for all non-interactive tmux calls, which resolves the issue. Note that the `console` command does not set this, as that is an interactive shell, and the user's terminfo should be passed. Pastebin log demonstrating the issue with ghostty terminal: https://pastebin.com/JeAVUfk4 Fixes: #4861 ## Type of change - [x] Bug fix (a change which fixes an issue). ## Checklist PR will not be merged until all steps are complete. - [x] This pull request links to an issue. - [x] This pull request uses the `develop` branch as its base. - [x] This pull request subject follows the Conventional Commits standard. - [x] This code follows the style guidelines of this project. - [x] I have performed a self-review of my code. - [x] I have checked that this code is commented where required. - [x] I have provided a detailed enough description of this PR. - [x] I have checked if documentation needs updating. ## Documentation If documentation does need updating either update it by creating a PR (preferred) or request a documentation update. - User docs: https://github.com/GameServerManagers/LinuxGSM-Docs - Dev docs: https://github.com/GameServerManagers/LinuxGSM-Dev-Docs **Thank you for your Pull Request!** --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>