[GH-ISSUE #3487] [FEATURE] - Not actually require postfix, or a way to override that #2363

Open
opened 2026-02-27 03:02:29 +03:00 by kerem · 2 comments
Owner

Originally created by @dkowis on GitHub (Jun 10, 2021).
Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/3487

User Story

As a homelab server runner, I want to not be required to install postfix, because I'm using a different sendmail provider.

I've got msmtp installed, which relays to my node that runs actual postfix to do heavier SMTP bits. having postfix installed also conflicts with this, and doesn't need to be there.

Basic info

  • Distro: Ubuntu 20.04
  • Game: Valheim (in this case, but it'd apply to anything I think)
  • Command: check-update
  • LinuxGSM version: v21.2.3

Further Information

It'd be nice if it would detect that it's got a sendmail command to do the sending with, and not explicitly require postfix. I think it's completely fine to suggest/require postfix if nothing is installed, because that makes it easy, but I've provided a sendmail command via another package, and I think my chef even removes postfix in favor of the msmtp package, and it's kinda frustrating :)

Further Reading

I could link to msmtp documentation I guess: https://marlam.de/msmtp/

Originally created by @dkowis on GitHub (Jun 10, 2021). Original GitHub issue: https://github.com/GameServerManagers/LinuxGSM/issues/3487 ## User Story As a homelab server runner, I want to not be required to install postfix, because I'm using a different `sendmail` provider. I've got `msmtp` installed, which relays to my node that runs actual postfix to do heavier SMTP bits. having postfix installed also conflicts with this, and doesn't need to be there. ## Basic info * **Distro:** Ubuntu 20.04 * **Game:** Valheim (in this case, but it'd apply to anything I think) * **Command:** check-update * **LinuxGSM version:** v21.2.3 ## Further Information It'd be nice if it would detect that it's got a `sendmail` command to do the sending with, and not explicitly require postfix. I think it's completely fine to suggest/require postfix if nothing is installed, because that makes it easy, but I've provided a `sendmail` command via another package, and I think my chef even removes postfix in favor of the msmtp package, and it's kinda frustrating :) ## Further Reading I could link to msmtp documentation I guess: https://marlam.de/msmtp/
Author
Owner

@dgibbs64 commented on GitHub (Jun 11, 2021):

Dependency checking needs a refactor. So this should be part of that

<!-- gh-comment-id:859527967 --> @dgibbs64 commented on GitHub (Jun 11, 2021): Dependency checking needs a refactor. So this should be part of that
Author
Owner

@dgibbs64 commented on GitHub (Sep 24, 2021):

The code exists but doesnt quite work for your case. I will rework this to improve email detection

	# Adds postfix to required dependencies if email alert is enabled.
	if [ "${emailalert}" == "on" ]; then
		if [ -f /usr/bin/mailx ]; then
			if [ -d /etc/exim4 ]; then
				array_deps_required+=( exim4 )
			elif [ -d /etc/sendmail ]; then
				array_deps_required+=( sendmail )
			elif [ "$(command -v dpkg-query 2>/dev/null)" ]; then
				array_deps_required+=( mailutils postfix )
			elif [ "$(command -v rpm 2>/dev/null)" ]; then
				array_deps_required+=( mailx postfix )
			fi
		else
			if [ "$(command -v dpkg-query 2>/dev/null)" ]; then
				array_deps_required+=( mailutils postfix )
			elif [ "$(command -v rpm 2>/dev/null)" ]; then
				array_deps_required+=( mailx postfix )
			fi
		fi
	fi
}```
<!-- gh-comment-id:926625964 --> @dgibbs64 commented on GitHub (Sep 24, 2021): The code exists but doesnt quite work for your case. I will rework this to improve email detection ```fn_deps_email(){ # Adds postfix to required dependencies if email alert is enabled. if [ "${emailalert}" == "on" ]; then if [ -f /usr/bin/mailx ]; then if [ -d /etc/exim4 ]; then array_deps_required+=( exim4 ) elif [ -d /etc/sendmail ]; then array_deps_required+=( sendmail ) elif [ "$(command -v dpkg-query 2>/dev/null)" ]; then array_deps_required+=( mailutils postfix ) elif [ "$(command -v rpm 2>/dev/null)" ]; then array_deps_required+=( mailx postfix ) fi else if [ "$(command -v dpkg-query 2>/dev/null)" ]; then array_deps_required+=( mailutils postfix ) elif [ "$(command -v rpm 2>/dev/null)" ]; then array_deps_required+=( mailx postfix ) fi fi fi }```
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#2363
No description provided.