[GH-ISSUE #4404] [Bug]: check_system_requirements.sh fails if game is not explicitly listed or if default RAM is not defined as an integer (currently set to 0.5) #2751

Closed
opened 2026-02-27 03:05:13 +03:00 by kerem · 5 comments
Owner

Originally created by @Pathas-Gamer on GitHub (Dec 4, 2023).
Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/4404

User story

As a user, I want Factorio to start. It currently does not. It fails with an error seemingly checking system RAM

Game

Factorio

Linux distro

Ubuntu 22.04

Command

command: start

Further information

It seems ramrequirementgb is supposed to be an integer according to the log output below. However, after all the if/elif for defining the RAM requirements for each game, the final condition sets this value to 0.5 - not an integer.

else
ramrequirementgb="0.5"
fi

If the game or engine has a minimum RAM Requirement, compare it to system's available RAM.

if [ "${ramrequirementgb}" ]; then
if [ "${physmemtotalgb}" -lt "${ramrequirementgb}" ]; then
fn_print_dots "Checking RAM"
fn_print_warn_nl "Checking RAM: ${ramrequirementgb}G required, ${physmemtotal} available"
echo "* ${gamename} server may fail to run or experience poor performance."
fn_sleep_time_5
fi
fi

NOTE: Verified the bug disappears by changing the last else/fi from 0.5 to 1. Either the variable needs to be redefined as something supporting non-integer values, or the defined minimum RAM needs to be changed to an integer.

Relevant log output

factoriouser@factorio:~$ ./fctrserver start
[ WARN ] Starting fctrserver: Default RCON password detected
/home/factoriouser/lgsm/modules/check_system_requirements.sh: line 66: [: 0.5: integer expression expected
[ FAIL ] Starting fctrserver: Unable to start TestServer

Steps to reproduce

  1. Install Factorio server
  2. Attempt to start server
  3. Observe error about check_system_requirements.sh line 66
Originally created by @Pathas-Gamer on GitHub (Dec 4, 2023). Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/4404 ### User story As a user, I want Factorio to start. It currently does not. It fails with an error seemingly checking system RAM ### Game Factorio ### Linux distro Ubuntu 22.04 ### Command command: start ### Further information It seems ramrequirementgb is supposed to be an integer according to the log output below. However, after all the if/elif for defining the RAM requirements for each game, the final condition sets this value to 0.5 - not an integer. else ramrequirementgb="0.5" fi # If the game or engine has a minimum RAM Requirement, compare it to system's available RAM. if [ "${ramrequirementgb}" ]; then if [ "${physmemtotalgb}" -lt "${ramrequirementgb}" ]; then fn_print_dots "Checking RAM" fn_print_warn_nl "Checking RAM: ${ramrequirementgb}G required, ${physmemtotal} available" echo "* ${gamename} server may fail to run or experience poor performance." fn_sleep_time_5 fi fi NOTE: Verified the bug disappears by changing the last else/fi from 0.5 to 1. Either the variable needs to be redefined as something supporting non-integer values, or the defined minimum RAM needs to be changed to an integer. ### Relevant log output ```shell factoriouser@factorio:~$ ./fctrserver start [ WARN ] Starting fctrserver: Default RCON password detected /home/factoriouser/lgsm/modules/check_system_requirements.sh: line 66: [: 0.5: integer expression expected [ FAIL ] Starting fctrserver: Unable to start TestServer ``` ### Steps to reproduce 1. Install Factorio server 2. Attempt to start server 3. Observe error about check_system_requirements.sh line 66
Author
Owner

@dgibbs64 commented on GitHub (Dec 4, 2023):

Thanks, I am aware of the bug with system requirements. I just forgot to fix it before release. However, the system requirements bug is different fro the factorio issue. You have 2 issues that are unrelated. The system requirements bug will not affect factorio from starting

<!-- gh-comment-id:1838120367 --> @dgibbs64 commented on GitHub (Dec 4, 2023): Thanks, I am aware of the bug with system requirements. I just forgot to fix it before release. However, the system requirements bug is different fro the factorio issue. You have 2 issues that are unrelated. The system requirements bug will not affect factorio from starting
Author
Owner

@Pathas-Gamer commented on GitHub (Dec 4, 2023):

So far with this Factorio installation I have only encountered this bug on
line 66 of the system requirements script. What’s the other one I should be
looking out for?

Thanks!!

On Mon, Dec 4, 2023 at 3:10 AM Daniel Gibbs @.***>
wrote:

Thanks, I am aware of the bug with system requirements. I just forgot to
fix it before release. However, the system requirements bug is different
fro the factorio issue. You have 2 issues that are unrelated


Reply to this email directly, view it on GitHub
https://github.com/GameServerManagers/LinuxGSM/issues/4404#issuecomment-1838120367,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3COCN6BTBQJPKDV4GBVYO3YHWHO7AVCNFSM6AAAAABAFMSRP2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZYGEZDAMZWG4
.
You are receiving this because you authored the thread.Message ID:
@.***>

<!-- gh-comment-id:1838654348 --> @Pathas-Gamer commented on GitHub (Dec 4, 2023): So far with this Factorio installation I have only encountered this bug on line 66 of the system requirements script. What’s the other one I should be looking out for? Thanks!! On Mon, Dec 4, 2023 at 3:10 AM Daniel Gibbs ***@***.***> wrote: > Thanks, I am aware of the bug with system requirements. I just forgot to > fix it before release. However, the system requirements bug is different > fro the factorio issue. You have 2 issues that are unrelated > > — > Reply to this email directly, view it on GitHub > <https://github.com/GameServerManagers/LinuxGSM/issues/4404#issuecomment-1838120367>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/A3COCN6BTBQJPKDV4GBVYO3YHWHO7AVCNFSM6AAAAABAFMSRP2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZYGEZDAMZWG4> > . > You are receiving this because you authored the thread.Message ID: > ***@***.***> >
Author
Owner

@FlorianLeChat commented on GitHub (Dec 6, 2023):

I get the same warning message with a Garry's Mod server without affecting its startup.

<!-- gh-comment-id:1842853842 --> @FlorianLeChat commented on GitHub (Dec 6, 2023): I get the same warning message with a Garry's Mod server without affecting its startup.
Author
Owner

@lugaamn commented on GitHub (Dec 8, 2023):

cs2@ubuntu:~$ ./cs2server start
[ .... ] Starting cs2server: /home/cs2/lgsm/modules/check_system_requirements.sh: line 66: [: 0.5: integer expression expected

<!-- gh-comment-id:1846707496 --> @lugaamn commented on GitHub (Dec 8, 2023): cs2@ubuntu:~$ ./cs2server start [ .... ] Starting cs2server: /home/cs2/lgsm/modules/check_system_requirements.sh: line 66: [: 0.5: integer expression expected
Author
Owner

@github-actions[bot] commented on GitHub (Mar 3, 2025):

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

<!-- gh-comment-id:2692983420 --> @github-actions[bot] commented on GitHub (Mar 3, 2025): This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Sign in to join this conversation.
No labels
Atomic
Epic
cannot reproduce
command: backup
command: console
command: debug
command: details
command: fast-dl
command: install
command: mods
command: monitor
command: post-details
command: restart
command: send
command: start
command: stop
command: update
command: update-lgsm
command: validate
command: wipe
distro: AlmaLinux
distro: Arch Linux
distro: CentOS
distro: Debian
distro: Fedora
distro: RedHat
distro: Rocky Linux
distro: Ubuntu
distro: openSUSE
engine: goldsrc
engine: source
game: 7 Days to Die
game: ARMA 3
game: Ark: Survival Evolved
game: Assetto Corsa
game: Avorion
game: BATTALION: Legacy
game: Barotrauma
game: Battalion 1944
game: Battlefield 1942
game: Black Mesa: Deathmatch
game: Blade Symphony
game: Call of Duty 2
game: Call of Duty 4
game: Call of Duty: United Offensive
game: Counter-Strike 1.6
game: Counter-Strike 2
game: Counter-Strike: Global Offensive
game: Counter-Strike: Source
game: Day of Infamy
game: Dayz
game: Death Match Classic
game: Don't Starve Together
game: ET: Legacy
game: Eco
game: Factorio
game: Factorio
game: Garry's Mod
game: Half-Life
game: Hurtword
game: Insurgecy
game: Insurgecy
game: Insurgency: Sandstorm
game: Just Cause 3
game: Killing Floor
game: Killing Floor 2
game: Left 4 Dead 2
game: Minecraft
game: Minecraft Bedrock
game: Mordhau
game: Multi Theft Auto
game: Mumble
game: Natural Selection 2
game: No More Room in Hell
game: Pavlov VR
game: Post Scriptum
game: Project Zomboid
game: Quake 3
game: QuakeWorld
game: Red Orchestra: Ostfront 41-45
game: Return to Castle Wolfenstein
game: Rising World
game: Rust
game: San Andreas Multiplayer
game: Satisfactory
game: Soldat
game: Soldier of Fortune 2
game: Squad
game: Squad 44
game: Starbound
game: Stationeers
game: Sven Co-op
game: Team Fortress 2
game: Teamspeak 3
game: Teeworlds
game: Terraria
game: The Front
game: Unreal Tournament 2004
game: Unreal Tournament 3
game: Unreal Tournament 99
game: Unturned
game: Valheim
game: Wurm Unlimited
game: Zombie Master Reborn
game: label missing
good first issue
help wanted
info: alerts
info: dependency
info: docker
info: docs
info: email
info: query
info: steamcmd
info: systemd
info: tmux
info: website
info: website
needs more info
outcome: duplicate
outcome: issue resolved
outcome: issue resolved
outcome: issue unresolved
outcome: pr accepted
outcome: pr rejected
outcome: unconfirmed
outcome: wontfix
outcome: wrong forum
potential-duplicate
priority
pull-request
type: bug
type: feature
type: feature
type: feature request
type: game server request
type: refactor
waiting response
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/LinuxGSM#2751
No description provided.