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

Fails to build on x86_64-unknown-netbsd #344

Closed
lnicola opened this issue Aug 3, 2021 · 6 comments
Closed

Fails to build on x86_64-unknown-netbsd #344

lnicola opened this issue Aug 3, 2021 · 6 comments

Comments

@lnicola
Copy link

lnicola commented Aug 3, 2021

$ git checkout 5.0.0-pre.11
Previous HEAD position was 43ea645 Add missing comma in "As used by:" list
HEAD is now at 71c57b4 Prepare 5.0.0-pre.11
$ cargo build --target x86_64-unknown-netbsd
Updating crates.io index
   Compiling kqueue-sys v1.0.2
error[E0308]: mismatched types
  --> ~/.cargo/registry/src/github.com-1ecc6299db9ec823/kqueue-sys-1.0.2/src/lib.rs:68:20
   |
68 |             udata: ptr::null_mut(),
   |                    ^^^^^^^^^^^^^^^ expected `isize`, found *-ptr
   |
   = note:     expected type `isize`
           found raw pointer `*mut _`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0308`.
error: could not compile `kqueue-sys`

To learn more, run the command again with --verbose.

Spotted in rust-lang/rust#87719 (comment).

@0xpr03
Copy link
Member

0xpr03 commented Aug 3, 2021

@xanderio can you have a look at this ?

@lnicola
Copy link
Author

lnicola commented Aug 3, 2021

Looks like it's a kqueue-sys bug.

@0xpr03
Copy link
Member

0xpr03 commented Aug 3, 2021

This was probably introduced by #335 though it seems to stem from upstream. I sadly can't test this. Edit: And 335 was checked on freebsd, not netbsd, so I wonder..

@lnicola
Copy link
Author

lnicola commented Aug 3, 2021

Filed https://gitlab.com/worr/rust-kqueue-sys/-/issues/1.

0xpr03 added a commit that referenced this issue Aug 12, 2021
Signed-off-by: Aron Heinecke <aron.heinecke@t-online.de>
@0xpr03
Copy link
Member

0xpr03 commented Aug 12, 2021

I've added a PR to pin the 1.0.3 as minimum, what I'm currently missing is a way to run this on github actions a test build.

@0xpr03
Copy link
Member

0xpr03 commented Aug 17, 2021

Released as 5.0.0-pre.12

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

No branches or pull requests

2 participants