Skip to content

release canaries: run each command under the step's bash, by path - #731

Merged
Sunrisepeak merged 1 commit into
mainfrom
fix/canary-bash-by-path
Sep 28, 2026
Merged

Sunrisepeak merged 1 commit into
mainfrom
fix/canary-bash-by-path

Conversation

@speak-agent

Copy link
Copy Markdown
Member

The first release run of the canaries (release run 36363585412, for 2026.9.28.2) failed every GalTranslPP command before building anything: release_canaries.py started bash by name, and a Windows program that does so gets System32\bash.exe, the WSL launcher ("Windows Subsystem for Linux has no installed distributions"), because the loader searches the system directory before PATH. The gate held: no tag was created.

  • The canary workflow names the bash that runs the step (CANARY_BASH, converted with cygpath -w on Windows), and the runner uses it; without it the runner keeps bash.
  • tests/scripts/test_release_canaries.py covers the argv, a failing command, a failing first command of a pipeline, a missing expectation, the unpin, and the repository's list; it runs in the Linux static checks.

Criterion: the canary workflow dispatched on this branch builds GalTranslPP on windows-2025 with the candidate. The release is dispatched again after this merges.

The first release run of the canaries (release run 36363585412) failed every
GalTranslPP command with "Windows Subsystem for Linux has no installed
distributions": release_canaries.py started `bash` by name, and a Windows
program that does so gets System32's WSL launcher, which the loader finds
before PATH. The workflow now names the bash running the step (CANARY_BASH, a
Windows path from cygpath on Windows) and the runner uses it. The runner's
tests (tests/scripts/test_release_canaries.py) run in the Linux static checks.
@Sunrisepeak
Sunrisepeak merged commit e251ec3 into main Sep 28, 2026
44 of 46 checks passed
Sunrisepeak pushed a commit that referenced this pull request Sep 28, 2026
…-review (#733)

The plan of the 2026-09-28 round is landed (revision 4). Section 8 records the departures from section 7, nine findings (F7: the lint's reading of the #729 siblings; F8: the release gate's first reading was of its own harness, #731; F9: a packaging revision reaches a consumer with its index), the before and after reading of every task, the three sandbox runs (27 ok and 0 failed for the new pair), GalTranslPP on Windows three ways, the index's full sweep with 2026.9.28.2, and a self-review. The CHANGELOG of 2026.9.28.2 names #731. Documentation only.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants