Skip to content

Conversation

@dgibbs64
Copy link
Member

@dgibbs64 dgibbs64 commented Nov 9, 2025

Description

This PR updates compatibilities of various game servers that no longer work on newer distros

Fixes #[issue]

Type of change

  • Bug fix (a change which fixes an issue).
  • New feature (a change which adds functionality).
  • New Server (new server added).
  • Refactor (restructures existing code).
  • Comment update (typo, spelling, explanation, examples, etc).

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.

Thank you for your Pull Request!

* Corrected the operating system version for `Vintage Story` from `ubuntu-22.04` to `ubuntu-24.04`.
* Ensures compatibility with the latest server requirements.
* Adjusted version checks for supported distributions.
* Added specific error messages for unsupported game titles based on OS version.
Copilot AI review requested due to automatic review settings November 9, 2025 23:07
@dgibbs64 dgibbs64 added this to the v25.3.0 milestone Nov 9, 2025
@dgibbs64 dgibbs64 merged commit 31031c6 into develop Nov 9, 2025
9 checks passed
@dgibbs64 dgibbs64 deleted the feature/ubuntu24 branch November 9, 2025 23:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates game compatibility and dependency requirements for LinuxGSM. The changes move distro detection earlier in the dependency checking process, add explicit version constraints for specific games on Ubuntu and Debian, remove obsolete SSL library dependencies for Sven Co-op, and update the supported distribution versions for Sven Co-op and Vintage Story to Ubuntu 24.04.

Key changes:

  • Adds distribution version checks to prevent installation of bf1942/bfv on Ubuntu > 22.04 or Debian > 12, and onset/btl on Ubuntu > 20.04 or Debian > 11
  • Removes libssl1.1:i386 dependency for Sven Co-op across all Ubuntu and Debian versions
  • Updates Sven Co-op and Vintage Story support to Ubuntu 24.04 in serverlist.csv

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

File Description
lgsm/modules/check_deps.sh Moves info_distro.sh call earlier and adds version compatibility checks for specific games
lgsm/data/ubuntu-*.csv Removes libssl1.1:i386 dependency for Sven Co-op across multiple Ubuntu versions
lgsm/data/debian-*.csv Removes libssl1.1:i386 dependency for Sven Co-op across Debian 9, 10, and 11
lgsm/data/serverlist.csv Updates supported Ubuntu version for Sven Co-op and Vintage Story from 20.04/22.04 to 24.04

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


info_distro.sh
# These titles are only supported up to Ubuntu 22.04 (Jammy) and Debian 12 (Bookworm).
if { [ "${distroid}" == "ubuntu" ] && dpkg --compare-versions "${distroversion}" "gt" "22.04"; } || { [ "${distroidlike}" == "debian" ] && dpkg --compare-versions "${distroversion}" "gt" "12"; }; then
Copy link

Copilot AI Nov 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent use of distroidlike instead of distroid for Debian check. The variable distroidlike represents derivative relationships (e.g., Ubuntu's distroidlike is "debian"), not the actual distro ID. For actual Debian installations, this should check distroid == \"debian\" to properly detect Debian systems. The current logic may fail to detect actual Debian installations.

Copilot uses AI. Check for mistakes.
fi

# These titles are only supported up to Ubuntu 20.04 and Debian 11 (and Debian-like derivatives).
if { [ "${distroid}" == "ubuntu" ] && dpkg --compare-versions "${distroversion}" "gt" "20.04"; } || { [ "${distroidlike}" == "debian" ] && dpkg --compare-versions "${distroversion}" "gt" "11"; }; then
Copy link

Copilot AI Nov 9, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent use of distroidlike instead of distroid for Debian check. The variable distroidlike represents derivative relationships (e.g., Ubuntu's distroidlike is "debian"), not the actual distro ID. For actual Debian installations, this should check distroid == \"debian\" to properly detect Debian systems. The current logic may fail to detect actual Debian installations.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants