fabi/Sylpheed
1
0
Fork 0
Code Issues 9 Pull Requests Actions Packages Projects Releases Wiki Activity

notes: build-reborn points at a source root that does not exist either

Browse Source 
Found while checking what "Reborn" names. build-reborn line 15 is
SRC="${PROJECT_DIR:-/work}/Syplheed-Reborn" -- transposed letters -- and no such
directory exists; the workspace is at /work itself. It fails immediately with
cd: /work/Syplheed-Reborn: No such file or directory, so the documented way to
run the disc-gated tests is broken in this container.

Records the direct alternative, setting SYLPHEED_DISC by hand, which is what this
session has actually been doing.

This is the second wrapper here pointing at a source root that does not exist --
build-canary has the same defect and blocks the audio tap. Worth checking a
wrapper s SRC before trusting that a green or a failure came from your own code.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QsEPXWVaEpyfudtR6re1Pd
This commit is contained in:
sylph-decoder
2026-08-29 18:47:35 +00:00
parent b53a947bd9
commit fe70a0cd27
1 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
docs/agents/CONTAINER-NOTES.md
View File

@@ -118,7 +118,19 @@ python3 tools/re-capture/gmem.py find hex:820af844 400
## Verifying your own work
* Reborn's disc-gated tests **self-skip** without `SYLPHEED_DISC`. A green run
with it unset means almost nothing. `build-reborn test` wires it up for you.
with it unset means almost nothing.
🔴 **But `build-reborn` does not work in this container (2026-08-29).** Line 15
is `SRC="${PROJECT_DIR:-/work}/Syplheed-Reborn"` — note the transposed letters —
and no such directory exists; the workspace is at **`/work`** itself. It fails
immediately with `cd: /work/Syplheed-Reborn: No such file or directory`, so the
documented way to run the disc-gated tests is broken.
✅ **Run them directly instead**, setting the variable yourself:
```bash
SYLPHEED_DISC=/disc cargo test -p sylpheed-formats --test <name>
```
⚠️ This is the **second** wrapper in this container pointing at a source root
that does not exist — `build-canary` has the same defect. Check a wrapper's
`SRC` before trusting that a green or a failure came from your code.
* Prefer a headless self-verify over "it compiles": `sylpheed-cli mesh render`,
`screen render`, `save info` all produce checkable artifacts.
* A Bevy system-parameter conflict is invisible to the type checker and panics