From 8055e8632820ed250e95e55d44da28235317c7c2 Mon Sep 17 00:00:00 2001
From: Daniel Gibbs <me@danielgibbs.co.uk>
Date: Sun, 19 Apr 2026 10:03:22 +0000
Subject: [PATCH] fix(install_config): replace cp -nv with explicit file
 existence check

Newer coreutils (Ubuntu 24.04+, Debian 13+) emit a portability warning
when using cp -n: 'behavior of -n is non-portable and may change in future'.

Replace all cp -nv usages with an explicit [ ! -f dest ] guard, which is
portable across all distros and preserves the same OK/SKIP/FAIL behaviour.
---
 lgsm/modules/install_config.sh | 33 ++++++++++++++++++++++++++++-----
 1 file changed, 28 insertions(+), 5 deletions(-)

diff --git a/lgsm/modules/install_config.sh b/lgsm/modules/install_config.sh
index 41d4ad866..888fe099a 100755
--- a/lgsm/modules/install_config.sh
+++ b/lgsm/modules/install_config.sh
@@ -34,8 +34,13 @@ fn_default_config_remote() {
 		if [ "${config}" == "${servercfgdefault}" ]; then
 			mkdir -p "${servercfgdir}"
 			echo -en "copying config file [ ${italic}${servercfgfullpath}${default} ]"
-			changes+=$(cp -nv "${lgsmdir}/config-default/config-game/${config}" "${servercfgfullpath}")
-			exitcode=$?
+			if [ ! -f "${servercfgfullpath}" ]; then
+				cp "${lgsmdir}/config-default/config-game/${config}" "${servercfgfullpath}"
+				exitcode=$?
+				[ "${exitcode}" -eq 0 ] && changes="copied"
+			else
+				exitcode=0
+			fi
 			if [ "${exitcode}" -ne 0 ]; then
 				fn_print_fail_eol_nl
 				fn_script_log_fail "copying config file ${servercfgfullpath}"
@@ -48,7 +53,13 @@ fn_default_config_remote() {
 		elif [ "${shortname}" == "arma3" ] && [ "${config}" == "${networkcfgdefault}" ]; then
 			mkdir -p "${servercfgdir}"
 			echo -en "copying config file [ ${italic}${networkcfgfullpath}${default} ]"
-			changes+=$(cp -nv "${lgsmdir}/config-default/config-game/${config}" "${networkcfgfullpath}")
+			if [ ! -f "${networkcfgfullpath}" ]; then
+				cp "${lgsmdir}/config-default/config-game/${config}" "${networkcfgfullpath}"
+				exitcode=$?
+				[ "${exitcode}" -eq 0 ] && changes="copied"
+			else
+				exitcode=0
+			fi
 			if [ "${exitcode}" -ne 0 ]; then
 				fn_print_fail_eol_nl
 				fn_script_log_fail "copying config file ${networkcfgdefault}"
@@ -60,7 +71,13 @@ fn_default_config_remote() {
 			fi
 		elif [ "${shortname}" == "dst" ] && [ "${config}" == "${clustercfgdefault}" ]; then
 			echo -en "copying config file [ ${italic}${clustercfgfullpath}${default} ]"
-			changes+=$(cp -nv "${lgsmdir}/config-default/config-game/${clustercfgdefault}" "${clustercfgfullpath}")
+			if [ ! -f "${clustercfgfullpath}" ]; then
+				cp "${lgsmdir}/config-default/config-game/${clustercfgdefault}" "${clustercfgfullpath}"
+				exitcode=$?
+				[ "${exitcode}" -eq 0 ] && changes="copied"
+			else
+				exitcode=0
+			fi
 			if [ "${exitcode}" -ne 0 ]; then
 				fn_print_fail_eol_nl
 				fn_script_log_fail "copying config file ${clustercfgfullpath}"
@@ -72,7 +89,13 @@ fn_default_config_remote() {
 			fi
 		else
 			echo -en "copying config file [ ${italic}${servercfgdir}/${config}${default} ]"
-			changes+=$(cp -nv "${lgsmdir}/config-default/config-game/${config}" "${servercfgdir}/${config}")
+			if [ ! -f "${servercfgdir}/${config}" ]; then
+				cp "${lgsmdir}/config-default/config-game/${config}" "${servercfgdir}/${config}"
+				exitcode=$?
+				[ "${exitcode}" -eq 0 ] && changes="copied"
+			else
+				exitcode=0
+			fi
 			if [ "${exitcode}" -ne 0 ]; then
 				fn_print_fail_eol_nl
 				fn_script_log_fail "copying config file ${servercfgdir}/${config}"