Skip to content

Commit 93865c3

Browse files
CopilotPDowney
andauthored
fix: print original path in realpath error message in run-install-step.sh
Agent-Logs-Url: https://github.com/EngineScript/EngineScript/sessions/0feba0da-e40b-4ac9-a0c8-feaf12e9414d Co-authored-by: PDowney <[email protected]>
1 parent 60ccf56 commit 93865c3

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,12 @@ Changes are organized by date, with the most recent changes listed first.
66

77
## 2026-04-10
88

9+
### 🐛 CI ERROR MESSAGE FIX
10+
11+
- Fixed misleading error message in `scripts/ci/run-install-step.sh`: when `realpath` fails to resolve the log directory path, the second debug line now prints `realpath input: $LOG_PARENT_DIR` (the original path that failed) instead of `realpath: $RESOLVED_LOG_PARENT` (which would always be empty at that point, making debugging harder).
12+
13+
14+
915
### 🔧 VHOST IMPORT CODE QUALITY IMPROVEMENTS
1016

1117
- Added explicit `return` statement at the end of `run_url_search_replace_if_present` in `scripts/functions/vhost/vhost-import.sh` to satisfy shell best-practice linting (SC2151/explicit-return warning).

scripts/ci/run-install-step.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ fi
6060

6161
if ! RESOLVED_LOG_PARENT="$(realpath "$LOG_PARENT_DIR" 2>/dev/null)"; then
6262
echo "Error: unable to resolve log directory path: $LOG_PARENT_DIR" >&2
63-
echo "realpath: $RESOLVED_LOG_PARENT" >&2
63+
echo "realpath input: $LOG_PARENT_DIR" >&2
6464
exit 1
6565
fi
6666

0 commit comments

Comments
 (0)