Skip to content

Conversation

@adegeo
Copy link
Contributor

@adegeo adegeo commented Dec 20, 2023

Fixes #36710

@gfoidl is this what you're thinking? You didn't really provide too much detail about where/what you wanted to see changed.


Internal previews

📄 File 🔗 Preview link
docs/standard/io/how-to-use-named-pipes-for-network-interprocess-communication.md How to: Use Named Pipes for Network Interprocess Communication

Copy link
Member

@gfoidl gfoidl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, that looks fine.

My point / assumption was that .NET NamedPipes uses linux's pipe under the covers.
As some failures arose, I digged into the code and saw that on linux UDS are actually used.

With this PR that makes it clear. Thanks!

@adegeo adegeo enabled auto-merge (squash) December 21, 2023 19:45
@adegeo
Copy link
Contributor Author

adegeo commented Dec 27, 2023

@tdykstra can you review? Thanks!

@adegeo adegeo disabled auto-merge December 27, 2023 16:57
Copy link
Contributor

@tdykstra tdykstra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@adegeo adegeo merged commit d0d6a8b into main Dec 29, 2023
@adegeo adegeo deleted the adegeo-36710 branch December 29, 2023 20:08
IEvangelist pushed a commit that referenced this pull request Dec 30, 2023
* Fix typography at sample code in fsharp snippets (#38913)

* Extract remarks for System.Globalization* (#38912)

* fix build suggestions (#38917)

* Remove snap reference from include file (#38902)

* clarify linux portions of http3/libmsquic (#38901)

* Add note about linux and IPC (#38849)

* Update package index with latest published versions (#38920)

* Update ScopedBackgroundService (#38880)

* Update ScopedBackgroundService.cs

* Update scoped-service.md

* Make it clear you can install multiple versions (#38919)

* Make it clear you can install multiple versions

* more clarifications

* fix link

* clarify greps (#38921)

* Add clarifications around Arm packages and Ubuntu (#38922)

* Clarify Arm

* Clarify Arm

---------

Co-authored-by: Takamasa Matsui <[email protected]>
Co-authored-by: Genevieve Warren <[email protected]>
Co-authored-by: Andy (Steve) De George <[email protected]>
Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Varorbc <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Named Pipes on Linux don't use FIFO named pipes, rather they use Unix-domain-sockets

5 participants