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

Update newlib socket types to FreeBSD #1477

Merged
merged 1 commit into from
Aug 17, 2019

Conversation

leo60228
Copy link
Contributor

No description provided.

@rust-highfive
Copy link

r? @gnzlbg

(rust_highfive has picked a reviewer for you, use r? to override)

@gnzlbg
Copy link
Contributor

gnzlbg commented Aug 17, 2019

These are breaking changes to all newlib targets, not just FreeBSD right ?

@leo60228
Copy link
Contributor Author

@gnzlbg As far as I'm aware, only me and @FenrirWolf are actually libc newlib.

@gnzlbg
Copy link
Contributor

gnzlbg commented Aug 17, 2019

ok, @bors: r+

@bors
Copy link
Contributor

bors commented Aug 17, 2019

📌 Commit eb287a1 has been approved by gnzlbg

@FenrirWolf
Copy link
Contributor

FenrirWolf commented Aug 17, 2019

@gnzlbg yeah, these newlib bindings were originally added to support a custom nintendo 3ds homebrew target, and now @leo60228 is using them for a similar toolchain for the nintendo switch. No idea if anyone else has ever tried using them.

Technically these changes will break networking for my target, but one way around that would be to put the old definitions in the arm subfolder and have the new ones under aarch64

@bors
Copy link
Contributor

bors commented Aug 17, 2019

⌛ Testing commit eb287a1 with merge c18f563...

bors added a commit that referenced this pull request Aug 17, 2019
Update newlib socket types to FreeBSD

None
@gnzlbg
Copy link
Contributor

gnzlbg commented Aug 17, 2019

How are you compiling with these? Do you think it would be possible for you to try to set up some docker containers so that we can run run-time tests for these?

@bors
Copy link
Contributor

bors commented Aug 17, 2019

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

@bors bors merged commit eb287a1 into rust-lang:master Aug 17, 2019
@leo60228
Copy link
Contributor Author

@gnzlbg I have a Docker image at rustyhorizon/docker, but I haven't been able to get binaries to run in an emulator. I think somebody was trying to set up a CI farm out of broken Switches.

bors added a commit that referenced this pull request Aug 19, 2019
Define newlib socket types by target arch

This is a follow-up to #1477. As mentioned in that PR, the primary (only?) users of these newlib definitions are myself and @leo60228. This PR makes it so that my arm-based 3DS target keeps using the old socket definitions while leo's aarch64-based Switch target uses the BSD versions.

The 3DS also has inverted definitions for `POLLHUP` and `POLLOUT` for whatever reason, so I figured I'd take care of that here too.
bors added a commit that referenced this pull request Aug 20, 2019
Define newlib socket types by target arch

This is a follow-up to #1477. As mentioned in that PR, the primary (only?) users of these newlib definitions are myself and @leo60228. This PR makes it so that my arm-based 3DS target keeps using the old socket definitions while leo's aarch64-based Switch target uses the BSD versions.

The 3DS also has inverted definitions for `POLLHUP` and `POLLOUT` for whatever reason, so I figured I'd take care of that here too.
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.

5 participants