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 missing link_name for getmntinfo #2586

Merged
merged 1 commit into from Dec 10, 2021

Conversation

GuillaumeGomez
Copy link
Member

@Amanieu
Copy link
Member

Amanieu commented Dec 10, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Dec 10, 2021

📌 Commit 3e601af has been approved by Amanieu

@GuillaumeGomez
Copy link
Member Author

Fixed format.

@Amanieu
Copy link
Member

Amanieu commented Dec 10, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Dec 10, 2021

📌 Commit 5dd94a0 has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Dec 10, 2021

⌛ Testing commit 5dd94a0 with merge 47c96a5...

bors added a commit that referenced this pull request Dec 10, 2021
Add missing link_name for getmntinfo

r? `@Amanieu`
@bors
Copy link
Contributor

bors commented Dec 10, 2021

💔 Test failed - checks-cirrus-freebsd-11

@GuillaumeGomez
Copy link
Member Author

@asomers Do you know what's going on in here by any chance?

@@ -4037,6 +4037,10 @@ extern "C" {
pub fn getfh(path: *const ::c_char, fhp: *mut fhandle_t) -> ::c_int;
pub fn lgetfh(path: *const ::c_char, fhp: *mut fhandle_t) -> ::c_int;
pub fn getfsstat(buf: *mut ::statfs, bufsize: ::c_long, mode: ::c_int) -> ::c_int;
#[cfg_attr(
all(target_os = "freebsd", freebsd11),
link_name = "getmnntinfo@FBSD_1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
link_name = "getmnntinfo@FBSD_1.0"
link_name = "getmntinfo@FBSD_1.0"

Copy link
Member Author

Choose a reason for hiding this comment

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

I feel very stupid right now... Thanks!

@Amanieu
Copy link
Member

Amanieu commented Dec 10, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Dec 10, 2021

📌 Commit 6192def has been approved by Amanieu

@bors
Copy link
Contributor

bors commented Dec 10, 2021

⌛ Testing commit 6192def with merge 720652a...

@bors
Copy link
Contributor

bors commented Dec 10, 2021

☀️ Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 720652a to master...

@bors bors merged commit 720652a into rust-lang:master Dec 10, 2021
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

5 participants