fix: resolve #795 Support pathname Unix-domain streams on Windows - #796
Closed
chenzeyan54-commits wants to merge 1 commit into
Closed
chenzeyan54-commits wants to merge 1 commit into
chenzeyan54-commits wants to merge 1 commit into
Conversation
The Windows local-address provider always returns -95, and the Windows branches of `_unix_socket` and `unix_remove` do the same. Consequently, portabl... Signed-off-by: chenzeyan54-commits <chenzeyan54-commits@users.noreply.github.com>
Member
|
Is the work done? It’s still a draft. |
Open
LunaStev
added a commit
to LunaStev/Wave
that referenced
this pull request
Oct 2, 2026
Use the Windows sockaddr_un layout and validate UTF-8 pathname bytes before opening sockets. Enable the existing stream operations and explicit removal, retaining native errors and closing sockets on failed setup. Preserve finite and infinite empty event waits through Sleep without calling WSAPoll with no descriptors. Cover native Windows targets and injected OS failures, including cleanup errors, byte boundaries, and timeout behavior. Follow up on wavefnd#796 and wavefnd#797; address wavefnd#795 and wavefnd#793. Signed-off-by: LunaStev <luna@lunastev.org>
4 tasks done
Member
|
The original contribution has been incorporated into #839, preserving the commit author and message. Follow-up changes complete the agreed Windows pathname socket behavior and add regression coverage. Closing this PR as superseded by #839, which is still open and awaiting validation and merge. Thank you for the contribution. |
LunaStev
added a commit
that referenced
this pull request
Oct 2, 2026
…#839) * Publish validated master builds through a rolling Nightly release Signed-off-by: LunaStev <luna@lunastev.org> * fix(#795): Support pathname Unix-domain streams on Windows The Windows local-address provider always returns -95, and the Windows branches of `_unix_socket` and `unix_remove` do the same. Consequently, portabl... Signed-off-by: chenzeyan54-commits <chenzeyan54-commits@users.noreply.github.com> (cherry picked from commit 950ae4f) Signed-off-by: LunaStev <luna@lunastev.org> * fix(windows): handle empty event sets without WSAPoll (cherry picked from commit dd90054) Signed-off-by: LunaStev <luna@lunastev.org> * style: format boundary regression test (cherry picked from commit a5520b1) Signed-off-by: LunaStev <luna@lunastev.org> * Complete Windows pathname streams and preserve empty wait timeouts Use the Windows sockaddr_un layout and validate UTF-8 pathname bytes before opening sockets. Enable the existing stream operations and explicit removal, retaining native errors and closing sockets on failed setup. Preserve finite and infinite empty event waits through Sleep without calling WSAPoll with no descriptors. Cover native Windows targets and injected OS failures, including cleanup errors, byte boundaries, and timeout behavior. Follow up on #796 and #797; address #795 and #793. Signed-off-by: LunaStev <luna@lunastev.org> * Fix static mock linking and make WASI host startup synchronous Link OS-boundary mocks without PIE, exercise fatal WASI exits with live host handles, and point the README build badge at ci.yml. Signed-off-by: LunaStev <luna@lunastev.org> --------- Signed-off-by: LunaStev <luna@lunastev.org> Signed-off-by: chenzeyan54-commits <chenzeyan54-commits@users.noreply.github.com> Co-authored-by: chenzeyan54-commits <chenzeyan54@gmail.com> Co-authored-by: tenkeren11 <332676327+Tenkeren11@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Fixes #795
Overview
The Windows local-address provider always returns -95, and the Windows branches of
_unix_socketandunix_removedo the same. Consequently, portabl...Key Changes
Verification