Skip to content

fix(windows): do not error on @process.spawn when there is no stdio - #565

Merged
Guest0x0 merged 1 commit into
mainfrom
fix-windows-no-stdio-spawn
Aug 21, 2026
Merged

fix(windows): do not error on @process.spawn when there is no stdio#565
Guest0x0 merged 1 commit into
mainfrom
fix-windows-no-stdio-spawn

Conversation

@Guest0x0

Copy link
Copy Markdown
Collaborator

By default, child process spawned by @process.spawn etc. inherit stdio channels of current process. On Windows, some programs such as GUI applications simply do not have stdio channel. Previously, @process.spawn will fail in this case, which is not desirable. This PR fixes the problem by tolerating stdio not present error when inheriting stdio channels in process spawning.

@coveralls

coveralls commented Aug 21, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 1049

Coverage remained the same at 77.099%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 4908
Covered Lines: 3784
Line Coverage: 77.1%
Coverage Strength: 12258.97 hits per line

💛 - Coveralls

Comment thread src/internal/event_loop/thread_pool.c
@Guest0x0
Guest0x0 force-pushed the fix-windows-no-stdio-spawn branch from 2664857 to 483821b Compare August 21, 2026 06:47
@Guest0x0
Guest0x0 merged commit 41fb5ca into main Aug 21, 2026
21 checks passed
@Guest0x0
Guest0x0 deleted the fix-windows-no-stdio-spawn branch August 21, 2026 07:35
@Guest0x0 Guest0x0 added the wasm-runtime-port-required The change in this PR need to be ported to WASM runtime label Aug 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasm-runtime-port-required The change in this PR need to be ported to WASM runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants