Twinned from https://github.com/cfsh/ninefold/issues/569 by tools/beads/import_github.py's reconcile pass.
Found while making Godot discovery platform-aware for #566. Left out of that PR deliberately: it is two concerns, and neither is on the merge path.
1. The glob is Windows-only. run_godot_tests.sh:18:
```bash GODOT="${GODOT_CONSOLE:-$(ls C:/Tools/Godot/*/*_console.exe 2>/dev/null | head -1 || true)}" ```
_console.exe is a Windows-subsystem artifact — Linux ships one binary that always writes to stdout — so on the WSL box this matches nothing and the script exits 2 with "set GODOT_CONSOLE", on a machine where Godot is installed and working at /opt/godot. Every other Godot lookup now goes through nflib.godot.find(), which handles both platforms; this is the last site that does not.
2. It is checked-in shell holding logic. Per notes/shell-policy.md a .sh may resolve paths and exec one thing; this one globs, branches on two conditions and emits a diagnostic. So the fix is probably not "add a Linux glob" but "make it Python", at which point it can just call nflib.godot.
Careful — two properties are pinned by tests and were paid for:
test_launchers.test_the_godot_lookup_tolerates_an_unmatched_glob asserts || true is on that line. Under pipefail an unmatched glob makes ls exit 2, an assignment takes its substitution's status, and set -e kills the script *before* the actionable message (#532 F5).test_launchers.test_a_missing_godot_prints_the_actionable_message_not_a_line_number simulates "not found" by rewriting the literal glob string with a regex. Any port has to give that test a new way to force the not-found path — and note /opt/godot now *exists* on the WSL box, so "point it at a real root and expect nothing" no longer works there.Unclaimed.
None.
No comments.