Skip to content

posix: implement fattach() and fdetach() #68464

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Feb 26, 2024

Conversation

moonlight83340
Copy link
Contributor

This is part of the See #51211 (RFC #51211).

fattach() and fdetach() are required as part of _XOPEN_STREAMS Option Group.

For more information, please refer to https://pubs.opengroup.org/onlinepubs/9699919799/functions/fattach.html

Fixes #66973
Fixes #66974

`fdetach()` and `fattach()` are required
as part of _XOPEN_STREAMS Option Group.

signed-off-by: Gaetan Perrot <gaetanperrotpro@gmail.com>
@moonlight83340 moonlight83340 marked this pull request as ready for review February 2, 2024 12:45
@zephyrbot zephyrbot added the area: POSIX POSIX API Library label Feb 2, 2024
@zephyrbot zephyrbot requested review from cfriedt and ycsin February 2, 2024 12:46
cfriedt
cfriedt previously approved these changes Feb 3, 2024
ycsin
ycsin previously approved these changes Feb 5, 2024
`fdetach()` and `fattach()` are now implemented, mark it so.

signed-off-by: Gaetan Perrot <gaetanperrotpro@gmail.com>
Add tests for `fdetach()` and `fattach()`

signed-off-by: Gaetan Perrot <gaetanperrotpro@gmail.com>
@moonlight83340
Copy link
Contributor Author

I add the fact that they are implement but will fail with ENOSYS because they are more likely a placeholder.

@moonlight83340 moonlight83340 requested a review from ycsin February 7, 2024 06:58
@fabiobaltieri fabiobaltieri added this to the v3.7.0 milestone Feb 7, 2024
@fabiobaltieri fabiobaltieri merged commit a4ea4e2 into zephyrproject-rtos:main Feb 26, 2024
@moonlight83340 moonlight83340 deleted the fattach_fdetach branch July 4, 2025 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: POSIX POSIX API Library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

posix: implement fdetach() posix: implement fattach()
5 participants