Skip to content

Commit f37b292

Browse files
committed
ci(fresh-install): drop redundant GITHUB_PATH append in container leg
The findutils fix unmasked a second, tumbleweed-only failure at the start of "Configure mcpp": OCI runtime exec failed: exec: "sh": executable file not found in $PATH exit code 127 opensuse/tumbleweed's image declares no PATH in its config. Appending a single dir to $GITHUB_PATH makes the runner exec every later step's `sh` with only that dir on PATH, so /usr/bin/sh becomes unfindable. The setup/install steps passed only because they ran before the append. Every container-matrix step already does `export PATH="$HOME/.xlings/subos/current/bin:$PATH"` in its own script, so the GITHUB_PATH append was redundant — and the sole cause of the corruption. Drop it. (The bare linux-fresh leg keeps its GITHUB_PATH: its later steps call mcpp directly without re-exporting.)
1 parent 38afc87 commit f37b292

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

.github/workflows/ci-fresh-install.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,13 @@ jobs:
173173
- name: Install xlings + mcpp
174174
run: |
175175
curl -fsSL https://raw.githubusercontent.com/openxlings/xlings/main/tools/other/quick_install.sh | bash -s v0.4.38
176-
echo "$HOME/.xlings/subos/current/bin" >> "$GITHUB_PATH"
176+
# Deliberately NOT writing to $GITHUB_PATH here. On container
177+
# images that declare no PATH in their config (opensuse/
178+
# tumbleweed), appending a single dir to GITHUB_PATH makes the
179+
# runner exec later steps' `sh` with only that dir on PATH —
180+
# `sh` (in /usr/bin) vanishes and the next step dies with
181+
# `exec: "sh": ... not found` / exit 127. Each step below already
182+
# exports PATH itself, so the append is redundant anyway.
177183
178184
- name: Configure mcpp
179185
run: |

0 commit comments

Comments
 (0)