Skip to content
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

liblibc should let you set errno #17714

Closed
ben0x539 opened this issue Oct 2, 2014 · 6 comments
Closed

liblibc should let you set errno #17714

ben0x539 opened this issue Oct 2, 2014 · 6 comments

Comments

@ben0x539
Copy link
Contributor

ben0x539 commented Oct 2, 2014

std::os::errno() doesn't come with a ::set_errno() variant, but this seems like fairly important functionality for interacting with C code.

There's https://github.com/bnoordhuis/rust-errno/blob/master/errno.rs, but not shipping with support for this is sort of a curious omission that belies Rust's appropriateness for slotting into C systems.
@ben0x539
Copy link
Contributor Author

ben0x539 commented Oct 2, 2014

Also that thing I linked is super old :v

@mahkoh
Copy link
Contributor

mahkoh commented Oct 2, 2014

#17265

@ben0x539
Copy link
Contributor Author

ben0x539 commented Oct 2, 2014

Didn't realize searching through issues doesn't list PRs anymore. Whoops!

Guess I might as well leave this open until the PR merges, thoug.

@ben0x539
Copy link
Contributor Author

ben0x539 commented Oct 2, 2014

Really we should expose something that returns a pointer to errno, if that's possible. C can do it...

@donkopotamus
Copy link
Contributor

#18642

@steveklabnik steveklabnik mentioned this issue Nov 7, 2015
Closed
@steveklabnik
Copy link
Member

moving to rust-lang/libc#47 now that this is an external crate.

lnicola pushed a commit to lnicola/rust that referenced this issue Jul 28, 2024
@bors
Auto merge of rust-lang#17714 - Throne3d:patch-1, r=Veykril
75fabf6 
docs: Reference cov_mark crate instead of test_utils/mark

Seems this was switched over in rust-lang/rust-analyzer#7922

I'm guessing this should skip the changelog because it's so minor, right? Reading through https://github.com/rust-lang/rust-analyzer/blob/master/docs/dev/style.md and trying to get up to speed here =)
RalfJung pushed a commit to RalfJung/rust that referenced this issue Aug 1, 2024
@bors
Auto merge of rust-lang#17714 - Throne3d:patch-1, r=Veykril
34dc7ac 
docs: Reference cov_mark crate instead of test_utils/mark

Seems this was switched over in rust-lang/rust-analyzer#7922

I'm guessing this should skip the changelog because it's so minor, right? Reading through https://github.com/rust-lang/rust-analyzer/blob/master/docs/dev/style.md and trying to get up to speed here =)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@ben0x539 @steveklabnik @mahkoh @donkopotamus