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

Rename the notbsd module to linux_like #1389

Merged
merged 1 commit into from
Jun 4, 2019
Merged

Conversation

gnzlbg
Copy link
Contributor

@gnzlbg gnzlbg commented Jun 3, 2019

No description provided.

@rust-highfive
Copy link

@gnzlbg: no appropriate reviewer found, use r? to override

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jun 4, 2019

@bors: r+

@bors
Copy link
Contributor

bors commented Jun 4, 2019

📌 Commit 8f098a3 has been approved by gnzlbg

@bors
Copy link
Contributor

bors commented Jun 4, 2019

⌛ Testing commit 8f098a3 with merge cf78b5d...

bors added a commit that referenced this pull request Jun 4, 2019

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Rename the notbsd module to linux_like
@bors
Copy link
Contributor

bors commented Jun 4, 2019

💔 Test failed - checks-travis

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Jun 4, 2019

@bors: retry

@bors
Copy link
Contributor

bors commented Jun 4, 2019

⌛ Testing commit 8f098a3 with merge ce7e3a7...

bors added a commit that referenced this pull request Jun 4, 2019

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Rename the notbsd module to linux_like
@bors
Copy link
Contributor

bors commented Jun 4, 2019

☀️ Test successful - checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing ce7e3a7 to master...

@kleimkuhler
Copy link

kleimkuhler commented Sep 22, 2019

@gnzlbg Hey I'm running into an error where notbsd::... is expected, but linux_like::... is found. I've replicated it in playground.

Any idea if this is an error in trying to make a SocketAddr or some version conflict?

Edit: I'm trying to make the SocketAddr here

pub struct SocketAddr {
    addr: libc::sockaddr_un,
    len: libc::socklen_t,
}

Edit 2: Sorry for the ping on this. I realize now I can't even make this type because the fields are private

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Sep 22, 2019

@kleimkuhler you should open an issue in rust-lang/rust since that looks like a bug in libstd.

Essentially, the types exposed by libc are not the same types exposed by libstd. Often, libstd has newtypes for them (e.g. pthread_t), but sometimes someone made a bug somewhere.

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Sep 22, 2019

Looking at the source, it appears that libstd does properly have a newtype here, the fields of that type are private, so you should never see a type error here, but a privacy error instead.

I think this is worth reporting as an error to the Rust compiler. The error message is confusing.

@gnzlbg
Copy link
Contributor Author

gnzlbg commented Sep 22, 2019

I've reported it here: rust-lang/rust#64684

@kleimkuhler
Copy link

@gnzlbg Oh okay thanks for the explanation on that--and reporting it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants