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.

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

add faccessat on illumos/solaris and euidaccess on solaris #2953

Merged
merged 1 commit into from Oct 9, 2022

Conversation

SteveLauC
Copy link
Contributor

@rust-highfive
Copy link

Some changes occurred in solarish module

cc @jclulow,@pfmooney

@rust-highfive
Copy link

r? @Amanieu

(rust-highfive has picked a reviewer for you, use r? to override)

@pfmooney
Copy link
Contributor

pfmooney commented Oct 9, 2022

The illumos side of things looks good to me.

@JohnTitor
Copy link
Member

Thanks both!
@bors r+

@bors
Copy link
Contributor

bors commented Oct 9, 2022

📌 Commit aa915ee has been approved by JohnTitor

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 9, 2022

⌛ Testing commit aa915ee with merge 0f5ee7a...

@bors
Copy link
Contributor

bors commented Oct 9, 2022

☀️ Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 0f5ee7a to master...

@bors bors merged commit 0f5ee7a into rust-lang:master Oct 9, 2022
@SteveLauC SteveLauC deleted the faccessat-on-illumos-and-solaris branch October 9, 2022 21:46
bors bot added a commit to nix-rust/nix that referenced this pull request Oct 10, 2022
1841: add faccessat on illumos r=asomers a=SteveLauC

This binding has been added in [libc#2953](rust-lang/libc#2953)

Co-authored-by: Steve Lau <stevelauc@outlook.com>
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.

None yet

6 participants