-
-
Notifications
You must be signed in to change notification settings - Fork 854
feat: update various game server distro compatibility's #4853
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
4a30a5c
dfefa51
e471d4b
21b5dbe
c8bdfe0
465162a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -351,6 +351,8 @@ if [ "${commandname}" == "INSTALL" ]; then | |
| fi | ||
| fi | ||
|
|
||
| info_distro.sh | ||
|
|
||
| # Will warn user if their distro is no longer supported by the vendor. | ||
| if [ -n "${distrosupport}" ]; then | ||
| if [ "${distrosupport}" == "unsupported" ]; then | ||
|
|
@@ -359,7 +361,23 @@ if [ -n "${distrosupport}" ]; then | |
| fi | ||
| fi | ||
|
|
||
| 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 | ||
| if [ "${shortname}" == "bf1942" ] || [ "${shortname}" == "bfv" ]; then | ||
| fn_print_failure_nl "${gamename} is not supported on ${distroname} (requires Ubuntu <= 22.04 or Debian <= 12)." | ||
| fn_script_log_fail "${gamename} is not supported on ${distroname}." | ||
| core_exit.sh | ||
| fi | ||
| 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 | ||
|
||
| if [ "${shortname}" == "onset" ] || [ "${shortname}" == "btl" ]; then | ||
| fn_print_failure_nl "${gamename} is not supported on ${distroname} (requires Ubuntu <= 20.04 or Debian <= 11)." | ||
| fn_script_log_fail "${gamename} is not supported on ${distroname}." | ||
| core_exit.sh | ||
| fi | ||
| fi | ||
|
|
||
| if [ ! -f "${tmpdir}/dependency-no-check.tmp" ] && [ ! -f "${datadir}/${distroid}-${distroversioncsv}.csv" ]; then | ||
| # Check that the distro dependency csv file exists. | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent use of
distroidlikeinstead ofdistroidfor Debian check. The variabledistroidlikerepresents derivative relationships (e.g., Ubuntu'sdistroidlikeis "debian"), not the actual distro ID. For actual Debian installations, this should checkdistroid == \"debian\"to properly detect Debian systems. The current logic may fail to detect actual Debian installations.