Skip to content

capture stdout from atags and FINALLY re-enable ci because of that#73

Merged
lun-4 merged 6 commits intomistressfrom
captured-stdout
Mar 15, 2025
Merged

capture stdout from atags and FINALLY re-enable ci because of that#73
lun-4 merged 6 commits intomistressfrom
captured-stdout

Conversation

@lun-4
Copy link
Owner

@lun-4 lun-4 commented Mar 15, 2025

problem is that from the new test runner protocol (around 0.12 or 0.13? don't remember) the tests started freezing, i hid it and instead ran the test binary directly, manually, as removing --listen=- made the tests work. after debugging it now (due to general zig 0.14 porting) it turns out that the protocol runs on stdin/stdout, and atags tests involve actually invoking the CLI utility code which will write to stdout.

to properly fix this we need to route stdout to somewhere else, done here via an "IOContext" but i may move that to Context.IOContext in the future so that all utilities can use this kind of code

in turn, zig build test can be finally re-enabled on CI without freezing

@lun-4 lun-4 merged commit 9ba92ab into mistress Mar 15, 2025
2 checks passed
@lun-4 lun-4 deleted the captured-stdout branch March 15, 2025 20:01
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.

1 participant