Skip to content

sysctlbynameZ does not return NameTooLong, drop dead prongs #24909

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

sysctlbynameZ does not return NameTooLong, drop dead prongs #24909

wants to merge 1 commit into from
+2 −4

Conversation

imomaliev
Copy link

For context read #24903

@imomaliev imomaliev mentioned this pull request Aug 19, 2025
Open
@imomaliev imomaliev marked this pull request as draft August 19, 2025 19:00
@imomaliev
Copy link
Author

imomaliev commented Aug 19, 2025 •
edited
Loading

Looks like I've rushed this. So we are back to discussion on 3 point

Last but not least, std.posix.sysctlbynameZ never returns NameTooLong. So either we should change the return type, or, as @andrewrk did in 3640303 we should add name_len handling. I did not find any mention of ENAMETOOLONG in manpages for freebsd/posix. So I am having a hard time understanding which way is correct here.

How should I proceed? Should I create a separate error union for sysctlbynameZ or should we just follow sysctl example and handle name_len after all?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@imomaliev