From 6a9edd4ee0d767f1b76df58ef422db74c32bbe8d Mon Sep 17 00:00:00 2001 From: Frank Date: Sat, 22 Aug 2026 22:36:04 -0700 Subject: [PATCH] deploy.sh: keep the relink comment free of a literal loop-var token The explanatory comment contained an unescaped loop-variable token, which the local shell (set -u) tried to expand while building the double-quoted remote script ('unit: unbound variable'). Reword the comment. --- scripts/deploy.sh | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/scripts/deploy.sh b/scripts/deploy.sh index 41d7538..439a556 100755 --- a/scripts/deploy.sh +++ b/scripts/deploy.sh @@ -219,9 +219,10 @@ fi echo '[4/5] Relinking LLM units and restarting LLM services...' for unit in ${LLM_UNITS[$TARGET]}; do - # No quotes around the path: ${unit} must expand on the remote (the path - # has no spaces). Quoting it would ship a literal ${unit} and create a - # dangling symlink (2026-08-22 bug). + # The loop variable must expand on the remote, so it is escaped here and + # the path is unquoted (it has no spaces). Quoting the source path would + # ship a literal loop-variable token and create a dangling symlink + # (the 2026-08-22 relink bug). ${SUDO} ln -sf ${REMOTE_DIR[$TARGET]}/scripts/\${unit}.service /etc/systemd/system/\${unit}.service done ${SUDO} systemctl daemon-reload