Skip to content

Add socket extensions for cygwin #139524

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Berrysoft
Copy link
Contributor

@Berrysoft Berrysoft commented Apr 8, 2025

r? @joboet

  • Abstract name uds addr
  • quickack
  • passcred

@rustbot rustbot added O-unix Operating system: Unix-like S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Apr 8, 2025
@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@Berrysoft Berrysoft marked this pull request as ready for review April 8, 2025 15:03
@Berrysoft
Copy link
Contributor Author

Berrysoft commented Apr 9, 2025

Some limitations:

  • Cannot connect the addr from recvfrom of a unix datagram socket.
    • It seems like a bug of cygwin.
    • A patch has been proposed.
    • UPDATE: patch merged. Now the test passes.
  • Cannot bind to abstract addr - not supported by cygwin.
  • Unbounded unix socket has an abstract local addr - a feature (autobind?) of cygwin.
  • Unix socket doesn't support recvmsg with control data. It fallbacks to recvmsg without control data.
  • Unix socket connect blocks until the server accepts it. See mailing list. I have to ignore some tests because of that.

@Berrysoft Berrysoft force-pushed the cygwin-socket-ext branch from 306b6a0 to e0b30bf Compare April 9, 2025 16:12
@jieyouxu jieyouxu added the O-cygwin Target: *-pc-cygwin label Apr 25, 2025
@bors
Copy link
Collaborator

bors commented Jun 29, 2025

☔ The latest upstream changes (presumably #143157) made this pull request unmergeable. Please resolve the merge conflicts.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
O-cygwin Target: *-pc-cygwin O-unix Operating system: Unix-like S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants