Libc 024

[j16r]

This is an attempt to get nix-rust compiling on FreeBSD. It' built on top of marjakm's fork which upgraded libc to 0.2.2.

[j16r]

Unfortunately none of the Travis builds are on FreeBSD and I definitely have sys::test_socket::test_scm_rights failing locally. Any tips on how I might add that builder?

[arthurprs]

Thank you for that. I'll use your fork meanwhile.

EDIT:

here's what I get

/root/.cargo/git/checkouts/nix-rust-bf0f6980b5e5fa55/master/src/sys/socket/consts.rs:208:44: 208:69 error: unresolved name `libc::IPV6_ADD_MEMBERSHIP` [E0425]
/root/.cargo/git/checkouts/nix-rust-bf0f6980b5e5fa55/master/src/sys/socket/consts.rs:208     pub const IPV6_ADD_MEMBERSHIP: c_int = libc::IPV6_ADD_MEMBERSHIP;
                                                                                                                                    ^~~~~~~~~~~~~~~~~~~~~~~~~
/root/.cargo/git/checkouts/nix-rust-bf0f6980b5e5fa55/master/src/sys/socket/consts.rs:208:44: 208:69 help: run `rustc --explain E0425` to see a detailed explanation
/root/.cargo/git/checkouts/nix-rust-bf0f6980b5e5fa55/master/src/sys/socket/consts.rs:210:45: 210:71 error: unresolved name `libc::IPV6_DROP_MEMBERSHIP` [E0425]
/root/.cargo/git/checkouts/nix-rust-bf0f6980b5e5fa55/master/src/sys/socket/consts.rs:210     pub const IPV6_DROP_MEMBERSHIP: c_int = libc::IPV6_DROP_MEMBERSHIP;
                                                                                                                                     ^~~~~~~~~~~~~~~~~~~~~~~~~~
/root/.cargo/git/checkouts/nix-rust-bf0f6980b5e5fa55/master/src/sys/socket/consts.rs:210:45: 210:71 help: run `rustc --explain E0425` to see a detailed explanation
   Compiling regex v0.1.44
/root/.cargo/registry/src/git.luolix.top-1ecc6299db9ec823/nix-0.3.9/src/unistd.rs:129:9: 129:22 error: unused import, #[deny(unused_imports)] on by default
/root/.cargo/registry/src/git.luolix.top-1ecc6299db9ec823/nix-0.3.9/src/unistd.rs:129     use errno::EINVAL;
                                                                                          ^~~~~~~~~~~~~
error: aborting due to previous error

[j16r]

What platform are you on @arthurprs ?

[arthurprs]

@j16r freebsd 10.2 amd64

[j16r]

Weird, I explicitly disabled that line for freebsd in commit 5df11f4. Are you cross compiling? Can you confirm you're building with target_os = freebsd? I'm building on the same platform and not getting your error, weird.

[arthurprs]

It looks like cargo was using an old commit, I added an explicit commit and it now fails with

root@:~/Desktop/floki # cargo test
    Updating git repository `https://github.com/j16r/nix-rust.git`
   Compiling nix v0.3.9
   Compiling nix v0.5.0-pre (https://github.com/j16r/nix-rust.git?rev=84d2c24d428a885e8d7500852b8b08714ab48598#84d2c24d)
   Compiling env_logger v0.3.2
   Compiling uuid v0.1.18
   Compiling twox-hash v0.1.1
/root/.cargo/registry/src/git.luolix.top-1ecc6299db9ec823/nix-0.3.9/src/unistd.rs:129:9: 129:22 error: unused import, #[deny(unused_imports)] on by default
/root/.cargo/registry/src/git.luolix.top-1ecc6299db9ec823/nix-0.3.9/src/unistd.rs:129     use errno::EINVAL;
                                                                                          ^~~~~~~~~~~~~
error: aborting due to previous error

[j16r]

Looks like you have two references to nix, one of them is pulling 0.3.9.

[arthurprs]

Oh, I see. It's mio, but I can't find a way to override that.

[arthurprs]

Ok I went around it using .cargo/config and it looks like your PR is fine :)

mio will need some adjustments before compiling on it though

[arthurprs]

@j16r I sent a PR to your fork in order to get mio working properly.

[j16r]

@arthurprs legend! I've merged it in.

[carllerche]

Thanks! Going to look at this now. Sorry for the delay.

[carllerche]

I'm trying to validate Mio working with this before merging.

[carllerche]

I am unsure how CI is passing given that it won't compile on my box... libc::IPV6_ADD_MEMBERSHIP doesn't seem to exist anymore for OS X (renamed).

[carllerche]

Oops, never mind. I was half way through a rebase

[carllerche]

Ok, let's see if Mio CI will pass with this: tokio-rs/mio#343

[arthurprs]

I guess we can close this now since pr343 is in

[carllerche]

Ah yes. Thanks

[j16r]

Hmm, I'm confused. pr343 is on mio. With this closed I'm still unable to use nix-rust on FreeBSD?

[kamalmarhubi]

@j16r this was merged in. I'm not sure what incantation @carllerche uses, but it results in him having to close PRs even when they are merged in. You can see the commits on this PR in the github history: https://github.com/carllerche/nix-rust/commits/master

[j16r]

Oh, I missed that, thanks.