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

OpenBSD & NetBSD support #185

Closed
wants to merge 1 commit into from
Closed

OpenBSD & NetBSD support #185

wants to merge 1 commit into from

Conversation

@worr
Copy link

worr commented Jan 1, 2018

Requires accounting for ancillary data in sending data. 63+ fd tests
are disabled, since this blocks on OpenBSD.

Also requires splitting sends and receives into separate threads during
testing.

Requires accounting for ancillary data in sending data. 63+ fd tests
are disabled, since this blocks on OpenBSD.

Also requires splitting sends and receives into separate threads during
testing.
@highfive
Copy link
Collaborator

highfive commented Jan 1, 2018

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @emilio (or someone else) soon.

// Empirically, we have to deduct 32 bytes from that.
const RESERVED_SIZE: usize = 32;
// Empirically, we have to deduct 72 bytes from that.
const RESERVED_SIZE: usize = 72;

This comment has been minimized.

@dlrobertson

dlrobertson Jan 4, 2018

Collaborator

Is this the same for linux, freebsd, netbsd, and openbsd?

@@ -94,11 +94,16 @@ fn medium_data() {
let data: Vec<u8> = (0..65536).map(|i| (i % 251) as u8).collect();
let data: &[u8] = &data[..];
let (tx, rx) = platform::channel().unwrap();
tx.send(data, vec![], vec![]).unwrap();
let thread = thread::spawn(move || {

This comment has been minimized.

@dlrobertson

dlrobertson Jan 4, 2018

Collaborator

why is thread::spawn needed here and in other tests?

@bors-servo
Copy link
Contributor

bors-servo commented Mar 7, 2018

The latest upstream changes (presumably #191) made this pull request unmergeable. Please resolve the merge conflicts.

@jdm
Copy link
Member

jdm commented Mar 7, 2018

Closing due to lack of response. We're merging basic changes that allow the crate to build on OpenBSD; if there are still changes required for running tests, we can accept a separate PR to address those.

@jdm jdm closed this Mar 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

6 participants
You can’t perform that action at this time.